Skip to content

Fedora 43 upgrade #2729

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
Open
bpradipt wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fedora 43 upgrade #2729

bpradipt wants to merge 2 commits into from

Conversation

@bpradipt
Copy link
Member

@bpradipt bpradipt commented Dec 16, 2025

Upgrading Fedora base to latest supported version till we have Ubuntu podvm image ready

@bpradipt bpradipt requested a review from a team as a code owner December 16, 2025 14:04
@bpradipt
Copy link
Member Author

bpradipt commented Dec 16, 2025

Depends on #2730
Once 2730 is merged, I'll update the commits to use the new base image

@bpradipt bpradipt marked this pull request as draft December 16, 2025 14:08
stevenhorsman
stevenhorsman reviewed Dec 16, 2025
View reviewed changes
@bpradipt bpradipt marked this pull request as ready for review December 17, 2025 06:13
@bpradipt
Copy link
Member Author

bpradipt commented Dec 17, 2025

Not sure why TestLibvirtCreateWithCpuAndMemRequestLimit is failing

@bpradipt
Copy link
Member Author

bpradipt commented Dec 17, 2025

The important failure is the s390x mkosi job.

Failed to enable unit: Unit cloud-init.service does not exist

@bpradipt bpradipt requested a review from mkulke December 17, 2025 11:06
@stevenhorsman
Copy link
Member

stevenhorsman commented Dec 17, 2025

Not sure why TestLibvirtCreateWithCpuAndMemRequestLimit is failing

I think this will be that with the new base image the memory requirements are slightly larger, so we probably need to bump them in DoTestPodWithCpuMemLimitsAndRequests(t, testEnv, assert, "100m", "100Mi", "200m", "1792Mi") to give us more headroom as we were close to the limit before.

stevenhorsman
stevenhorsman reviewed Dec 17, 2025
View reviewed changes
mkulke
mkulke reviewed Dec 17, 2025
View reviewed changes
@bpradipt
Copy link
Member Author

bpradipt commented Jan 13, 2026

@stevenhorsman @mkulke and others, do we want to upgrade to 43 for the upcoming release ?

@stevenhorsman
Copy link
Member

stevenhorsman commented Jan 13, 2026

@stevenhorsman @mkulke and others, do we want to upgrade to 43 for the upcoming release ?

I was hoping that we could move to ubuntu instead, but I don't think there will be time for that to happen. I'm not sure we have a way to get the fedora 43 upgrade working though either?

@mkulke
Copy link
Collaborator

mkulke commented Jan 13, 2026

@stevenhorsman @mkulke and others, do we want to upgrade to 43 for the upcoming release ?

I'd go with bumping fedora for this release and then updating it in the next, I don't expect ubuntu to work without hickups. we can announce the plan in the release notes.

@bpradipt
podvm-mkosi: bump up to Fedora 43
b65df3d 
Latest supported release

Signed-off-by: Pradipta Banerjee <[email protected]>
@bpradipt
podvm-mkosi: blacklist vmgenid_plaform_driver_init
4452e0c 
Kernels after 6.14 are failing to boot on AWS SNP instances and
blacklisting vmgenid was found as a workaround.

A proper fix is posted upstream - https://www.spinics.net/lists/kernel/msg5976520.html

Fixes - confidential-containers#2691

Signed-off-by: Pradipta Banerjee <[email protected]>
@bpradipt bpradipt requested review from ldoktor and mkulke January 21, 2026 14:34
@bpradipt
Copy link
Member Author

bpradipt commented Jan 21, 2026

I have split the kernel cli change to a separate commit

@mkulke
Copy link
Collaborator

mkulke commented Jan 21, 2026

I have split the kernel cli change to a separate commit

a PR would be better (if the fix is urgent), since there is probably more work required for s390x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stevenhorsman stevenhorsman stevenhorsman left review comments

@mkulke mkulke Awaiting requested review from mkulke

@ldoktor ldoktor Awaiting requested review from ldoktor

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

test_e2e_docker test_e2e_libvirt Run Libvirt e2e tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bpradipt @stevenhorsman @mkulke @ldoktor