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

fedora-40 image refresh and naughty pattern for 7139 #7140

Merged
merged 2 commits into from
Nov 26, 2024

Conversation

@mvollmer
Copy link
Member Author

This should also trigger in fedora-40 already. Let's make an image right away...

@mvollmer mvollmer added the bot label Nov 25, 2024
@cockpituous cockpituous changed the title naughty: New pattern for 7139 WIP: ddbba30bf864: [no-test] naughty: New pattern for 7139 Nov 25, 2024
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: ddbba30bf864: [no-test] naughty: New pattern for 7139 naughty: New pattern for 7139 Nov 25, 2024
@cockpituous
Copy link
Contributor

@@ -0,0 +1,3 @@
# testBasic (__main__.TestHistoryMetrics.testBasic)
Copy link
Member

Choose a reason for hiding this comment

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

Please copy that to fedora-41 as well, see #7123

@martinpitt martinpitt mentioned this pull request Nov 25, 2024
2 tasks
@martinpitt
Copy link
Member

(Copied from the parallel image refresh #7126 -- there is no reason to believe that this refresh magically fixes any of these):

Phew, this is a big one. There's the python crashes from #21301, files needs a pixel update due to the new cockpit version enabling "open in terminal", sub-man causes an oops in the same manner as in #7123, and composer failed on an internal npm mirror failure (retried that one).

@mvollmer
Copy link
Member Author

(Copied from the parallel image refresh #7126 -- there is no reason to believe that this refresh magically fixes any of these):

Oops, I wasn't aware yet of that PR... sorry.

@martinpitt
Copy link
Member

TestMultiCPU.testCPUUsage not caught by naughty?

@mvollmer
Copy link
Member Author

TestMultiCPU.testCPUUsage not caught by naughty?

My mistake. Man, I am bad at multi-tasking...

@martinpitt martinpitt changed the title naughty: New pattern for 7139 fedora-40 image refresh and naughty pattern for 7139 Nov 25, 2024
@cockpituous cockpituous changed the title fedora-40 image refresh and naughty pattern for 7139 WIP: c6ac3dd0e86a: [no-test] fedora-40 image refresh and naughty pattern for 7139 Nov 26, 2024
@martinpitt martinpitt changed the title WIP: c6ac3dd0e86a: [no-test] fedora-40 image refresh and naughty pattern for 7139 fedora-40 image refresh and naughty pattern for 7139 Nov 26, 2024
@cockpituous cockpituous changed the title fedora-40 image refresh and naughty pattern for 7139 WIP: b2a370617910: [no-test] fedora-40 image refresh and naughty pattern for 7139 Nov 26, 2024
@martinpitt
Copy link
Member

martinpitt commented Nov 26, 2024

sudo dnf info cockpit-bridge shows 329.1 now, so refreshing. And yup, this picked up the new version

@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: b2a370617910: [no-test] fedora-40 image refresh and naughty pattern for 7139 fedora-40 image refresh and naughty pattern for 7139 Nov 26, 2024
@cockpituous
Copy link
Contributor

@martinpitt
Copy link
Member

Argh, still that machines failure. @jelly and I collided here, I sent cockpit-project/cockpit-machines#1925 to clean that up. Tested locally.

@martinpitt
Copy link
Member

cockpit-project/cockpit-machines#1925 landed, so retrying the machines tests. cockpit and -files only fail the pixel tests. After machines goes green, I'll land the pixel tests and then do a final rebase to land this fully green -- otherwise we'll ruin the flake statistics. It's evening and fairly quiet, so I think we can afford to do it this way.

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.

🚀 exhale

Thanks @mvollmer ! Great teamwork!

@martinpitt martinpitt merged commit 6ef6649 into cockpit-project:main Nov 26, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Python core dumps with metrics tests
3 participants