kubevirt / kubevirtci

Contains cluster definitions and client tools to quickly spin up and destroy ephemeral and scalable k8s and ocp clusters for testing
Apache License 2.0
79 stars 119 forks source link

Add Fedora 40 containerdisk #1202

Open acardace opened 2 months ago

acardace commented 2 months ago

The new image provides an updated kernel which is needed for memory hotplug testing as virtio-mem requires the guest kernel to be 5.16+ at least.

What this PR does / why we need it: Add Fedora 40 to help with memory hotplug testing.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.

Release note:

Bump fedora-with-test-tooling image to Fedora 40
kubevirt-bot commented 2 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign ormergi for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[cluster-provision/images/vm-image-builder/OWNERS](https://github.com/kubevirt/kubevirtci/blob/main/cluster-provision/images/vm-image-builder/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
acardace commented 2 months ago

/cc @brianmcarey @dhiller

acardace commented 2 months ago

/retest

brianmcarey commented 2 months ago

It would be worth building a image and creating a test PR against kubevirt as this will likely break a number of network tests which I hit issues with previously trying to update this image to f39. https://github.com/kubevirt/kubevirt/pull/11419

I can try and get an image built for you.

brianmcarey commented 2 months ago

Running tests here - https://github.com/kubevirt/kubevirt/pull/12105

acardace commented 2 months ago

@brianmcarey thank you!

kubevirt-bot commented 2 months ago

@acardace: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
check-up-kind-1.27-vgpu c79c2407218770786bff77d4d7e361a89f602feb link false /test check-up-kind-1.27-vgpu
Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).