You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "brbzull0 (via GitHub)" <gi...@apache.org> on 2026/02/13 12:30:15 UTC
[PR] Fix log_retention test: use correct test run variable [trafficserver]
brbzull0 opened a new pull request, #12884:
URL: https://github.com/apache/trafficserver/pull/12884
This was failing on my box.
Test 7 ('Get the log to rotate.') was calling MakeCurlCommandMulti on test.tr (the test run from the TestLogRetention constructor) instead of the newly created tr. This left the new test run with no process defined, causing the test to fail with 'List came back empty.'
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix log_retention test: use correct test run variable [trafficserver]
Posted by "brbzull0 (via GitHub)" <gi...@apache.org>.
brbzull0 commented on PR #12884:
URL: https://github.com/apache/trafficserver/pull/12884#issuecomment-3896960125
@bneradt: This seems to be ok now , not sure if we haven't detect the issue because of the initial skip, but anyways. Works for me now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix log_retention test: use correct test run variable [trafficserver]
Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt commented on PR #12884:
URL: https://github.com/apache/trafficserver/pull/12884#issuecomment-3898110223
> @bneradt: This seems to be ok now , not sure if we haven't detect the issue because of the initial skip, but anyways. Works for me now.
Yep. This test is skipped in CI because it has timing issues.
Thanks for the fix!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix log_retention test: use correct test run variable [trafficserver]
Posted by "brbzull0 (via GitHub)" <gi...@apache.org>.
brbzull0 merged PR #12884:
URL: https://github.com/apache/trafficserver/pull/12884
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org