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

CI VMs: bump to f41 #24485

Merged
merged 3 commits into from
Nov 6, 2024
Merged

Conversation

edsantiago
Copy link
Member

@edsantiago edsantiago commented Nov 6, 2024

Built in containers/automation_images#392

Signed-off-by: Ed Santiago [email protected]

None

Copy link
Contributor

openshift-ci bot commented Nov 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: edsantiago

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

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 6, 2024
@Luap99
Copy link
Member

Luap99 commented Nov 6, 2024

FYI I just remembered d03e8ff so that should be reverted to actually test f40 not just f41

@edsantiago edsantiago force-pushed the test-f41 branch 2 times, most recently from baaa5da to 01812fb Compare November 6, 2024 17:11
@Luap99
Copy link
Member

Luap99 commented Nov 6, 2024

I guess we need to revert #24236 (or we just skip the test f40 part for a week or so until we get newer kernels) I am fine with whatever you prefer.

Signed-off-by: Ed Santiago <[email protected]>
This reverts commit d03e8ff.

Signed-off-by: Ed Santiago <[email protected]>
Temporary, until we get CI VMs with kernel 6.11.6.

I've lost track of where this is being discussed.

This reverts commit 7f836df.

Signed-off-by: Ed Santiago <[email protected]>
@edsantiago edsantiago changed the title DO NOT MERGE: test f41 VMs CI VMs: bump to f41 Nov 6, 2024
@edsantiago
Copy link
Member Author

All checkpoint and restore container out of and into pod tests are now running and passing on f41 and rawhide with crun 1.18.2. This closes my concern with #24327.

@mheon
Copy link
Member

mheon commented Nov 6, 2024

LGTM

1 similar comment
@TomSweeneyRedHat
Copy link
Member

LGTM

@TomSweeneyRedHat
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 6, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit b209474 into containers:main Nov 6, 2024
86 checks passed
@edsantiago edsantiago deleted the test-f41 branch November 7, 2024 12:29
@Luap99 Luap99 added the 5.3 label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.3 approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants