Skip to content

Test: add nameserver to proxy_serve_stale test #9059

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
maskit merged 2 commits into from
Aug 29, 2022
Merged

Test: add nameserver to proxy_serve_stale test #9059

maskit merged 2 commits into from
Aug 29, 2022

Conversation

@maskit
Copy link
Member

@maskit maskit commented Aug 25, 2022

No description provided.

@maskit maskit added the AuTest label Aug 25, 2022
@maskit maskit added this to the 10.0.0 milestone Aug 25, 2022
@maskit maskit self-assigned this Aug 25, 2022
ywkaras
ywkaras previously approved these changes Aug 25, 2022
View reviewed changes
@ywkaras ywkaras dismissed their stale review August 25, 2022 14:07

What for autopep8 format changes.

@maskit
Copy link
Member Author

maskit commented Aug 26, 2022 •
edited
Loading

[approve ci autest]

post-early-return faild.

@maskit
Copy link
Member Author

maskit commented Aug 26, 2022

[approve ci autest]

remap_load_missing_failure failed.

Reason: Returned Value -2 != 33
     file /tmp/sandbox/remap_load_missing_failure/ts/log/diags.log : Diags log file diags.log should not contain errors - Failed
        Reason: Cannot read /tmp/sandbox/remap_load_missing_failure/ts/log/diags.log: [Errno 2] No such file or directory: '/tmp/sandbox/remap_load_missing_failure/ts/log/diags.log'
     file /tmp/sandbox/remap_load_missing_failure/ts/log/diags.log : Diags log file diags.log should not contain errors - Failed

@maskit
Copy link
Member Author

maskit commented Aug 26, 2022

[approve ci autest]

1 similar comment
@maskit
Copy link
Member Author

maskit commented Aug 26, 2022

[approve ci autest]

@maskit maskit merged commit 44f2eaf into apache:master Aug 29, 2022
@zwoop
Copy link
Contributor

zwoop commented Aug 29, 2022

@maskit This has merge conflicts, like because of #8953. If you want this in 9.2.x, can you make a branch PR for 9.2.x please?

@zwoop zwoop removed this from the 10.0.0 milestone Aug 29, 2022
maskit added a commit to maskit/trafficserver that referenced this pull request Aug 29, 2022
@maskit
Test: add nameserver to proxy_serve_stale test (apache#9059)
c2d84d0 
* Test: add nameserver to proxy_serve_stale test

* autopep8

(cherry picked from commit 44f2eaf)

 Conflicts:
	tests/gold_tests/proxy_protocol/proxy_serve_stale.test.py
@maskit
Copy link
Member Author

maskit commented Aug 29, 2022

@zwoop I just made the PR for 9.2.x #9070

zwoop pushed a commit that referenced this pull request Aug 31, 2022
@maskit
Test: add nameserver to proxy_serve_stale test (#9059) (#9070)
f7b40be 
* Test: add nameserver to proxy_serve_stale test

* autopep8

(cherry picked from commit 44f2eaf)

 Conflicts:
	tests/gold_tests/proxy_protocol/proxy_serve_stale.test.py
@zwoop zwoop added this to the 10.0.0 milestone Jan 17, 2023
masaori335 pushed a commit to masaori335/trafficserver that referenced this pull request Feb 21, 2023
@zwoop
Merge remote-tracking branch 'asf/9.2.x' into 9.2.x
e905405 
* asf/9.2.x:
  Updated ChangeLog
  Adds efficient IP range matching to HRW conditions (apache#9031)
  Test: add nameserver to proxy_serve_stale test (apache#9059) (apache#9070)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ywkaras ywkaras ywkaras approved these changes

Assignees

@maskit maskit

Labels

AuTest

Projects

None yet

Milestone

10.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@maskit @zwoop @ywkaras