-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Ensure that dependencies are added to the operatorList for locally cached images (issue 18259) #18269
Conversation
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/6e0ea539a1a9e89/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/c7c5f7714d511d0/output.txt |
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. Thank you.
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/6e0ea539a1a9e89/output.txt Total script time: 28.63 mins
Image differences available at: http://54.241.84.105:8877/6e0ea539a1a9e89/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/c7c5f7714d511d0/output.txt Total script time: 44.11 mins
Image differences available at: http://54.193.163.58:8877/c7c5f7714d511d0/reftest-analyzer.html#web=eq.log |
…ched images (issue 18259)
b6b8464
to
22ca7d5
Compare
Missed one case... /botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/4f8a8bd7a5fea7f/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/205590cf8f15436/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/4f8a8bd7a5fea7f/output.txt Total script time: 28.68 mins
Image differences available at: http://54.241.84.105:8877/4f8a8bd7a5fea7f/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/205590cf8f15436/output.txt Total script time: 42.02 mins
Image differences available at: http://54.193.163.58:8877/205590cf8f15436/reftest-analyzer.html#web=eq.log |
Locally I get less "severe" movement when testing on Windows, perhaps this is worse with In any case, given that this patch ought to be correct are we OK with the ref-test movement in the latest round of testing? |
I have checked locally that this indeed solves the issue. I have also checked the last round of reference tests and didn't find anything that was either noticeable with the naked eye or that was unexpected movement, and given that I agree that this patch should be correct I think we can do this. Thanks! |
/botio makeref |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @timvandermeij received. Current queue size: 0 Live output at: http://54.193.163.58:8877/018642573633292/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_makeref from @timvandermeij received. Current queue size: 0 Live output at: http://54.241.84.105:8877/2645d88b795f3f7/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/2645d88b795f3f7/output.txt Total script time: 19.88 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/018642573633292/output.txt Total script time: 25.13 mins
|
No description provided.