-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Can't merge PR due to Chrome failing stability check and Firefox just timing out #5395
Comments
FWIW, I'm going to rebase the PR and repush to see if that changes anything because I'd really like to land this PR today. |
The PR in question is #3330, right? |
Yes, that's right |
(And it turns out I can't update the PR now. Not sure why. Does it get locked while the travis jobs are running?) |
Simple OWNERS change in #5394 is also being blocked. |
Just to be clear: this behavior is unrelated to ChromeDriver timeouts. The referenced Chrome job ran to completion and reported a failure due to unstable test results. |
@jugglinmike uh, I linked to the wrong issue for Chrome. #5408, which is the same issue as with Firefox |
Closing this since it hasn't come up since April, perhaps because Chrome failures aren't blocking. @JKereliuk has fixed lots of things in ChromeDriver and there's a new release soon, so any future failures might not be the same anyway. @lukebjerring at some point we should probably make Chrome failures blocking again, any thoughts on how to track issues like this to be able to enable it with confidence that we won't break everyone's workflows? |
Despite several retriggers I can't get this build to complete:
https://travis-ci.org/w3c/web-platform-tests/builds/219138173
The Firefox job mostly just sits there saying with no suggesting that it might ever end:
The Chrome job kept failing the stability check although I notice on this latest trigger it is now just doing:
I guess it will keep doing that until we run out of integers.
A build from the same PR with nearly the same test content yesterday, however, ran with no problems.
CC: @jugglinmike @bobholt
The text was updated successfully, but these errors were encountered: