-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
Skip failing FBF tests in Google Chrome when running makeref
on the bots
#11510
Skip failing FBF tests in Google Chrome when running makeref
on the bots
#11510
Conversation
… bots Note that this will still allow the FBF tests to run locally, and also on the bots when invoked with `test`/`browsertest` (to not lose all the FBF test-coverage), but will no longer prevent `makeref` from running successfully on the bots.
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.215.176.217:8877/17df7dae3782e5a/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/79bbd19bbcbba54/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.215.176.217:8877/17df7dae3782e5a/output.txt Total script time: 2.19 mins
Image differences available at: http://54.215.176.217:8877/17df7dae3782e5a/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/79bbd19bbcbba54/output.txt Total script time: 19.63 mins
Image differences available at: http://54.67.70.0:8877/79bbd19bbcbba54/reftest-analyzer.html#web=eq.log |
Let's try this... |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.215.176.217:8877/8b6450c2d07c38f/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_makeref from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/41b76b9689b1ccc/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.215.176.217:8877/8b6450c2d07c38f/output.txt Total script time: 2.53 mins
|
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/41b76b9689b1ccc/output.txt Total script time: 18.10 mins
|
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/c86cc108caf9ee0/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.215.176.217:8877/031eebd916d6d4f/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.215.176.217:8877/031eebd916d6d4f/output.txt Total script time: 2.12 mins
Image differences available at: http://54.215.176.217:8877/031eebd916d6d4f/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/c86cc108caf9ee0/output.txt Total script time: 19.57 mins
Image differences available at: http://54.67.70.0:8877/c86cc108caf9ee0/reftest-analyzer.html#web=eq.log |
The Windows bot seem to need a restart, or something... The Linux bot mostly works now, but unfortunately there's a number of Google Chrome test failures that seem intermittent. However, we might just have to live with this and simply disregard those failures when looking at the test logs!? |
/botio-windows makeref |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @brendandahl received. Current queue size: 0 Live output at: http://54.215.176.217:8877/922d526d0b914ba/output.txt |
From: Bot.io (Windows)SuccessFull output at http://54.215.176.217:8877/922d526d0b914ba/output.txt Total script time: 24.36 mins
|
/botio-windows test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.215.176.217:8877/22bbfea3b02da45/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.215.176.217:8877/22bbfea3b02da45/output.txt Total script time: 26.49 mins
Image differences available at: http://54.215.176.217:8877/22bbfea3b02da45/reftest-analyzer.html#web=eq.log |
It appears chrome on windows and linux is no longer reproducible. Last time we were able to extract the rendering that had an issue and file a bug with chromium (which I think they fixed). |
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.215.176.217:8877/83da448af34b58e/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/1ad1f70ae5804c2/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.215.176.217:8877/83da448af34b58e/output.txt Total script time: 26.21 mins
Image differences available at: http://54.215.176.217:8877/83da448af34b58e/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/1ad1f70ae5804c2/output.txt Total script time: 60.00 mins |
/botio-linux test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/6c0f88ebf69f9cb/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/6c0f88ebf69f9cb/output.txt Total script time: 14.80 mins
Image differences available at: http://54.67.70.0:8877/6c0f88ebf69f9cb/reftest-analyzer.html#web=eq.log |
/botio-linux makeref Trying out headless mode to see if it's stable. |
From: Bot.io (Linux m4)ReceivedCommand cmd_makeref from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/21a9df5ba9a3069/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/21a9df5ba9a3069/output.txt Total script time: 29.30 mins
|
Since Google Chrome is so inconsistent now, should we just disable it entirely for now so we can at least continue development, and try to enable it later on again using a follow-up issue? |
Unfortunately that didn't seem to help :-(
Either that, or we continue to run tests in Chrome and simply disregard these intermittent failures when looking at the results (although I suppose that this could increase the risk of actual regressions sneaking past a reviewer). |
I noticed that the Linux bot seems down right now since I cannot reach previews or reference image comparisons. How about we do the the following?
We could replace step 2 with only disabling the FBF tests as done in this patch, but then we'll have to ignore the intermittent failures which might make it harder to see what actually failed. However, one could argue that we then at least run most of the tests in Chrome. I'm on the fence about this step, so what do you think @brendandahl? |
I think we try to leave chrome chrome enabled for now and see how bad the intermittents are. If we're spending too much time validating tests then we can disable. |
/botio-linux makeref |
From: Bot.io (Linux m4)ReceivedCommand cmd_makeref from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/b56556d37a42104/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/b56556d37a42104/output.txt Total script time: 18.11 mins
|
/botio-linux test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/fde8c42f6858c2d/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/fde8c42f6858c2d/output.txt Total script time: 19.53 mins
Image differences available at: http://54.67.70.0:8877/fde8c42f6858c2d/reftest-analyzer.html#web=eq.log |
I agree that this approach is probably best. I'll make a follow-up ticket for the remaining intermittents and for re-enabling the FBF tests once Chrome stabilizes again. |
Note that this will still allow the FBF tests to run locally, and also on the bots when invoked with
test
/browsertest
(to not lose all the FBF test-coverage), but will no longer preventmakeref
from running successfully on the bots.