bneradt commented on PR #9664: URL: https://github.com/apache/trafficserver/pull/9664#issuecomment-1530787381
> @bneradt - actually, they've all failed and are now successful only because of retries. Exactly, some passed because of a retry. This is what I notice: * Your first run for this PR had some shutdown test failures. * Your second had a shutdown failure which I analyzed and commented upon above. * You have three or so other PRs you filed today which have had clean runs without any retries. My observation is that it seems like this patch is causing issues with some of the shutdown tests (fatal.test.py, emergency.test.py, etc.). I'm not sure why that's the case, but I don't want us to just re-run autest a bunch of times, get this merged in once all the tests pass by happenstance, then destabilize those tests for the rest of CI. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
