-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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.
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
Emit error if api is not stopped and exit code is greater than zero #22224
Conversation
💚 Build Succeeded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
💔 Build Failed |
We might need to revert this - @silne30 believes this is the cause for the ESCONNRESET's and the failed snapshots. |
Reverting this will break the Windows tests |
Can I get some more information on what ESCONNRESET error is? |
I think I originally had this under an if statement for the windows platform so I can do that if that helps. |
A partial fix for when chromedriver exists abnormally on windows in headless mode. There may still be a problem with killing processes on windows, but that error may be less frequent.
Issue: #22094
Part of issue: #21315