Skip to content
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

test: Update more pixel references for Chromium 96 #17051

Conversation

mvollmer
Copy link
Member

The known issue bots/#2877 was fixed and now we run more of the
TestStorageStratis.testBasic test and need to update one more pixel
test for version 96.

@mvollmer
Copy link
Member Author

Once this is green with the image from cockpit-project/bots#2960, we can just merge this since the changed reference image is not used with the current image.

@mvollmer mvollmer force-pushed the test-adjust-more-pixels-for-chrome-96 branch from ac473de to 36f8d9e Compare February 25, 2022 09:15
@mvollmer mvollmer added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Feb 25, 2022
The known issue bots/cockpit-project#2877 was fixed and now we run more of the
TestStorageStratis.testBasic test and need to update one more pixel
test for version 96.
@mvollmer mvollmer force-pushed the test-adjust-more-pixels-for-chrome-96 branch from 36f8d9e to d8a4cbb Compare February 25, 2022 09:16
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pixel diff

Hah, thanks!

@mvollmer mvollmer temporarily deployed to cockpit-dist February 25, 2022 09:23 Inactive
@mvollmer
Copy link
Member Author

Oops, triggered with the wrong bots PR.. please ignore fedora-35@bots#2969

@martinpitt martinpitt merged commit 1a10a83 into cockpit-project:main Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-test For doc/workflow changes, or experiments which don't need a full CI run,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants