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

feat: update bluefin stable kernel pin to 6.12.9 #2168

Merged
merged 2 commits into from
Jan 29, 2025
Merged

Conversation

bsherman
Copy link
Contributor

@bsherman bsherman commented Jan 29, 2025

Note: 6.12.9 is the current coreos-testing kernel and is known to have fixed the AMD suspend bug.

We're switching the "bluefin:stable" builds to use this kernel since it allows us to ship not-broken kernel and not-broken nvidia.

We don't have a pre-existing build for Fedora 40, so we can't ship a fix for "bluefin:gts" at this time.

Partial fix for #2157

This fixes bluefin:stable but not bluefin:gts

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Note: 6.12.9 is the current coreos-testing kernel and is known to have
fixed the AMD suspend bug.

We're switching the "bluefin:stable" builds to use this kernel since it
allows us to ship not-broken kernel and not-broken nvidia.

We don't have a pre-existing build for Fedora 40, so we can't ship a fix
for "bluefin:gts" at this time.
@bsherman
Copy link
Contributor Author

@castrojo maybe this PR should include a disabling of the GTS build until we have the officicial updated kernel?

@bsherman bsherman requested review from p5 and m2Giles January 29, 2025 01:45

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@bsherman bsherman marked this pull request as ready for review January 29, 2025 01:56
@bsherman bsherman requested a review from castrojo as a code owner January 29, 2025 01:56
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. enhancement New feature or request labels Jan 29, 2025
@bsherman
Copy link
Contributor Author

Looks like this is building what we expect for stable... a 6.12.9 kernel with nvidia 570 driver.

Copy link
Member

@p5 p5 left a comment

Choose a reason for hiding this comment

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

Thank you!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 29, 2025
@bsherman bsherman enabled auto-merge January 29, 2025 01:59
@bsherman
Copy link
Contributor Author

Thank you!

Thank YOU, @p5 , for noticing the stable images were still pulling old versions of the nvidia kmod!

@bsherman bsherman added this pull request to the merge queue Jan 29, 2025
Merged via the queue into main with commit 74e1527 Jan 29, 2025
36 checks passed
@bsherman bsherman deleted the stable-testing-6.12.9 branch January 29, 2025 03:11
bsherman added a commit that referenced this pull request Jan 29, 2025
- updates GTS kernel pin to 6.12.9
- re-enables weekly GTS builds disabled in PR #2168
- restores stable builds to use of `coreos-stable` kernel/akmods
bsherman added a commit that referenced this pull request Feb 26, 2025
We had pinned this in #2168 but we can unpin to get to newer kernels
again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants