You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/05/26 16:22:58 UTC
[GitHub] [trafficserver] brbzull0 opened a new pull request, #8869: Autest: Increase the delay time after a config reload rpc all was done.
brbzull0 opened a new pull request, #8869:
URL: https://github.com/apache/trafficserver/pull/8869
This particular test `remap_reload` seems to be failing randomly, it could be the delay time was too short, setting it to 10s now.
Let's see if this fixes https://github.com/apache/trafficserver/issues/8860
If not, I'll be happy to revert this back.
--
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
[GitHub] [trafficserver] brbzull0 closed pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
brbzull0 closed pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
URL: https://github.com/apache/trafficserver/pull/8869
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1138756703
> Thank you for working on this. Let's run the autest in here a few times (maybe 4?). If the test passes each of those times, let's merge it in.
sound good to me! thanks.
--
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
[GitHub] [trafficserver] bneradt commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1138924764
[approve ci autest]
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1139638036
[approve ci autest]
--
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
[GitHub] [trafficserver] bneradt commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1138755233
Thank you for working on this. Let's run the autest in here a few times (maybe 4?). If the test passes each of those times, let's merge it in.
--
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
[GitHub] [trafficserver] bneradt commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1138996576
[approve ci autest]
--
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
[GitHub] [trafficserver] bneradt commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1138882409
[approve ci autest]
--
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
[GitHub] [trafficserver] bneradt commented on pull request #8869: Autest: Increase the delay time after a config reload rpc all was done.
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8869:
URL: https://github.com/apache/trafficserver/pull/8869#issuecomment-1139988712
Reading through your patch in this PR made me realize that rather than waiting a certain amount of time for the reload to finish, we can instead await a log message that indicates that the reload has finished. Playing around with it a bit, I came up with this patch:
https://github.com/apache/trafficserver/pull/8872
Any thoughts?
--
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