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.

Sign up for GitHub

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

Jump to bottom

Image refresh for fedora-41 #7123

Merged
merged 3 commits into from
Nov 26, 2024
Merged

Image refresh for fedora-41 #7123

merged 3 commits into from
Nov 26, 2024

Conversation

cockpituous
Copy link
Contributor

@cockpituous cockpituous commented Nov 23, 2024 •
edited by martinpitt
Loading

@github-actions github-actions bot added the bot label Nov 22, 2024
@cockpituous cockpituous changed the title Image refresh for fedora-41 WIP: 78e2bb79864b: [no-test] Image refresh for fedora-41 Nov 22, 2024
@cockpituous cockpituous changed the title WIP: 78e2bb79864b: [no-test] Image refresh for fedora-41 Image refresh for fedora-41 Nov 22, 2024
@cockpituous
Copy link
Contributor

Failed. Log: https://cockpit-logs.us-east-1.linodeobjects.com/image-refresh-fedora-41-ea843220-20241122-224143/log.html

@cockpituous cockpituous changed the title Image refresh for fedora-41 WIP: cb4f57dd3a08: [no-test] Image refresh for fedora-41 Nov 23, 2024
@cockpituous
Copy link
Contributor

image-refresh fedora-41 done: https://github.com/cockpit-project/bots/commits/image-refresh-fedora-41-20241123-092150

cockpituous pushed a commit that referenced this pull request Nov 23, 2024
images: Update fedora-41 image
7bb7874 
Closes #7123
@cockpituous cockpituous force-pushed the image-refresh-fedora-41-20241123-092150 branch from b18dd7b to 7bb7874 Compare November 23, 2024 09:21
@cockpituous cockpituous changed the title WIP: cb4f57dd3a08: [no-test] Image refresh for fedora-41 Image refresh for fedora-41 Nov 23, 2024
@cockpituous
Copy link
Contributor

Success. Log: https://cockpit-logs.us-east-1.linodeobjects.com/image-refresh-fedora-41-ea843220-20241123-090320/log.html

@martinpitt
Copy link
Member

The metrics failures are the same python crashes as in #21301.

sub-man crashes, but it's not obvious why/how -- there's no crash in the js log.

This was referenced Nov 24, 2024
Closed
Closed
@martinpitt
Copy link
Member

martinpitt commented Nov 25, 2024 •
edited
Loading

I cannot reproduce these failures locally at all, neither with the old nor the new services image. Retrying.

Even a full TEST_OS=fedora-41 test/run from a clean tree passes locally 🤷‍♂️

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

I sent candlepin/subscription-manager-cockpit#84 to clean this up for the future. But we should put back cockpit.resolve() for now.

@mvollmer mvollmer force-pushed the image-refresh-fedora-41-20241123-092150 branch from 6806224 to 912f69f Compare November 25, 2024 14:45
@martinpitt
Copy link
Member

https://bodhi.fedoraproject.org/updates/FEDORA-2024-cbf666ab6a landed, trying a refresh.

@cockpituous cockpituous changed the title Image refresh for fedora-41 WIP: 502cf60d63b6: [no-test] Image refresh for fedora-41 Nov 26, 2024
cockpituous pushed a commit that referenced this pull request Nov 26, 2024
images: Update fedora-41 image
931c34f 
Closes #7123
@cockpituous
Copy link
Contributor

image-refresh fedora-41 done: https://github.com/cockpit-project/bots/commits/image-refresh-fedora-41-20241126-124533

@cockpituous cockpituous changed the title WIP: 502cf60d63b6: [no-test] Image refresh for fedora-41 Image refresh for fedora-41 Nov 26, 2024
@cockpituous
Copy link
Contributor

Success. Log: https://cockpit-logs.us-east-1.linodeobjects.com/image-refresh-fedora-41-912f69f0-20241126-122738/log.html

martinpitt pushed a commit that referenced this pull request Nov 26, 2024
@martinpitt
images: Update fedora-41 image
4771f8b 
Closes #7123
@martinpitt martinpitt force-pushed the image-refresh-fedora-41-20241123-092150 branch from 912f69f to 4771f8b Compare November 26, 2024 12:52
@martinpitt
Copy link
Member

Meh, more NFS fallout. But the storage ones should already be covered by #7137, why aren't they?

machines failures are half NFS and half cockpit-project/cockpit-machines#1914 (hopefully), retrying to confirm.

mvollmer and others added 3 commits November 26, 2024 14:42
@mvollmer @martinpitt
naughty: Add patterns for 7139
7ec7b84
@martinpitt
images: Update fedora-41 image
861b4d0 
Closes #7123
@martinpitt
naughty: Adjust #7137 pattern for Fedora 41
5ab0c77 
systemd in Fedora 41 changed the message. Also, some tests like
`TestSystemInfo.testCryptoPolicies` only show the failure without the
stack trace, so drop the glob. The "Test completed, but found unexpected
journal messages" match protects us from hiding more severe failures.
@martinpitt martinpitt force-pushed the image-refresh-fedora-41-20241123-092150 branch from 4771f8b to 5ab0c77 Compare November 26, 2024 13:49
@martinpitt
Copy link
Member

Apparently cockpit-project/cockpit-machines#1914 wasn't sufficient, it still fails. 🤯

@martinpitt martinpitt mentioned this pull request Nov 26, 2024
Merged
@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 martinpitt self-assigned this Nov 26, 2024
@martinpitt
Copy link
Member

Phew...

@martinpitt martinpitt merged commit d470b2c into main Nov 26, 2024
21 checks passed
martinpitt pushed a commit that referenced this pull request Nov 26, 2024
@martinpitt
images: Update fedora-41 image
58f28a7 
Closes #7123
@martinpitt martinpitt deleted the image-refresh-fedora-41-20241123-092150 branch November 26, 2024 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinpitt martinpitt martinpitt approved these changes

Assignees

@martinpitt martinpitt

Labels
bot no-test
Projects
Pilot tasks
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cockpituous @martinpitt @mvollmer