Remove the pinning of CentOS Stream kernel version #1080

brianmcarey · 2023-09-06T08:36:45Z

The latest kernel should have the shutdown issue resolved.

Includes update to CentOS Stream 9 image to 20230821.0

The latest kernel should have the shutdown issue resolved. Signed-off-by: Brian Carey <bcarey@redhat.com>

kubevirt-bot · 2023-09-06T08:36:48Z

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

brianmcarey · 2023-09-06T08:37:02Z

/test check-provision-k8s-1.27

akalenyu · 2023-09-06T10:38:42Z

Aren't we using an older centos image though?

kubevirtci/cluster-provision/centos9/version

Line 1 in 9bf8c8d

20221219.0

brianmcarey · 2023-09-06T12:01:50Z

Aren't we using an older centos image though?

kubevirtci/cluster-provision/centos9/version

Line 1 in 9bf8c8d

20221219.0

Yes you are right. I will try to upgrade the image too.

The kernel is always latest as it comes from here though - https://github.com/kubevirt/kubevirtci/pull/1080/files#diff-b5f457593536f8e24487be424871b5403701dde944795ba4d06c73fbb29c17edR31

Signed-off-by: Brian Carey <bcarey@redhat.com>

brianmcarey · 2023-09-06T12:03:54Z

/test check-provision-k8s-1.27

akalenyu · 2023-09-06T12:10:13Z

Aren't we using an older centos image though?

kubevirtci/cluster-provision/centos9/version

Line 1 in 9bf8c8d

20221219.0

Yes you are right. I will try to upgrade the image too.

The kernel is always latest as it comes from here though - https://github.com/kubevirt/kubevirtci/pull/1080/files#diff-b5f457593536f8e24487be424871b5403701dde944795ba4d06c73fbb29c17edR31

Are you sure? I was sitting on CDI main and we had 5.14.0-289.el9.x86_64 on the node
and kubevirtci version https://github.com/kubevirt/kubevirtci/releases/tag/2303311039-054c0ed

brianmcarey · 2023-09-06T12:15:59Z

Aren't we using an older centos image though?

kubevirtci/cluster-provision/centos9/version

Line 1 in 9bf8c8d

20221219.0

Yes you are right. I will try to upgrade the image too.
The kernel is always latest as it comes from here though - https://github.com/kubevirt/kubevirtci/pull/1080/files#diff-b5f457593536f8e24487be424871b5403701dde944795ba4d06c73fbb29c17edR31

Are you sure? I was sitting on CDI main and we had 5.14.0-289.el9.x86_64 on the node and kubevirtci version https://github.com/kubevirt/kubevirtci/releases/tag/2303311039-054c0ed

Sorry latest kernel at publishing time. 5.14.0-289.el9.x86_64 would of been the latest when that tag was published.

akalenyu · 2023-09-06T12:20:06Z

Aren't we using an older centos image though?

kubevirtci/cluster-provision/centos9/version

Line 1 in 9bf8c8d

20221219.0

Yes you are right. I will try to upgrade the image too.
The kernel is always latest as it comes from here though - https://github.com/kubevirt/kubevirtci/pull/1080/files#diff-b5f457593536f8e24487be424871b5403701dde944795ba4d06c73fbb29c17edR31

Are you sure? I was sitting on CDI main and we had 5.14.0-289.el9.x86_64 on the node and kubevirtci version https://github.com/kubevirt/kubevirtci/releases/tag/2303311039-054c0ed

Sorry latest kernel at publishing time. 5.14.0-289.el9.x86_64 would of been the latest when that tag was published.

Ohh right my bad

dhiller · 2023-09-06T17:56:34Z

cluster-provision/centos9/Dockerfile

@@ -27,11 +27,7 @@ ENV CENTOS_URL https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Str

 RUN /download_box.sh ${CENTOS_URL}

-# Pin to earlier versions of vmlinuz & initrd.img due to issue with latest


I think we should include this information inside some troubleshooting document, including a reference to the commit, so that we can apply this again if required.

This could also be done for other workarounds we had to do.

brianmcarey · 2023-09-07T09:07:14Z

/test check-provision-k8s-1.27

brianmcarey · 2023-09-08T13:18:02Z

/test check-provision-k8s-1.27

brianmcarey · 2023-09-11T18:34:24Z

/test check-provision-k8s-1.27

brianmcarey · 2023-09-12T10:18:53Z

/test check-provision-k8s-1.27

brianmcarey · 2023-09-12T20:31:43Z

/test check-provision-k8s-1.27

brianmcarey · 2023-09-13T07:13:34Z

/cc @xpivarc

This needs to be merged to unblock the publish of kubevirtci.

kubevirt-bot · 2023-09-13T07:30:03Z

@brianmcarey: 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	`23b9a35`	link	false	`/test check-up-kind-1.27-vgpu`

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

brianmcarey

@xpivarc thank you!

/approve

kubevirt-bot · 2023-09-13T07:34:48Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brianmcarey

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:

~~OWNERS~~ [brianmcarey]

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

[ece7e87 Remove the pinning of CentOS Stream kernel version](kubevirt/kubevirtci#1080) [52483da Downgrade k8s-1.27 to 1.27.1](kubevirt/kubevirtci#1077) [7a80619 Bump docker dependency](kubevirt/kubevirtci#1067) [beb5872 Increase default disk size to avoid node disk pressure](kubevirt/kubevirtci#1072) [27b06cb Pin to earlier versions of vmlinuz and initrd.img due to issue with](kubevirt/kubevirtci#1075) ```release-note NONE ``` Signed-off-by: kubevirt-bot <kubevirtbot@redhat.com>

Remove the pinning of CentOS Stream kernel version

9bf8c8d

The latest kernel should have the shutdown issue resolved. Signed-off-by: Brian Carey <bcarey@redhat.com>

kubevirt-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS labels Sep 6, 2023

Update CentOS Stream 9 image to 20230821.0

23b9a35

Signed-off-by: Brian Carey <bcarey@redhat.com>

kubevirt-bot added size/S and removed size/XS labels Sep 6, 2023

dhiller reviewed Sep 6, 2023

View reviewed changes

brianmcarey marked this pull request as ready for review September 13, 2023 07:12

kubevirt-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 13, 2023

kubevirt-bot requested review from dhiller and oshoval September 13, 2023 07:12

kubevirt-bot requested a review from xpivarc September 13, 2023 07:13

xpivarc approved these changes Sep 13, 2023

View reviewed changes

kubevirt-bot assigned xpivarc Sep 13, 2023

kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 13, 2023

brianmcarey commented Sep 13, 2023

View reviewed changes

kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 13, 2023

kubevirt-bot merged commit ece7e87 into kubevirt:main Sep 13, 2023
8 of 9 checks passed

kubevirt-bot mentioned this pull request Sep 13, 2023

Bump kubevirtci kubevirt/kubevirt#10408

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove the pinning of CentOS Stream kernel version #1080

Remove the pinning of CentOS Stream kernel version #1080

brianmcarey commented Sep 6, 2023 •

edited

Loading

kubevirt-bot commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023 •

edited

Loading

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023

dhiller Sep 6, 2023

brianmcarey commented Sep 7, 2023

brianmcarey commented Sep 8, 2023

brianmcarey commented Sep 11, 2023

brianmcarey commented Sep 12, 2023

brianmcarey commented Sep 12, 2023

brianmcarey commented Sep 13, 2023

kubevirt-bot commented Sep 13, 2023

brianmcarey left a comment

kubevirt-bot commented Sep 13, 2023

		@@ -27,11 +27,7 @@ ENV CENTOS_URL https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Str

		RUN /download_box.sh ${CENTOS_URL}

		# Pin to earlier versions of vmlinuz & initrd.img due to issue with latest

Remove the pinning of CentOS Stream kernel version #1080

Remove the pinning of CentOS Stream kernel version #1080

Conversation

brianmcarey commented Sep 6, 2023 • edited Loading

kubevirt-bot commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023 • edited Loading

brianmcarey commented Sep 6, 2023

akalenyu commented Sep 6, 2023

dhiller Sep 6, 2023

Choose a reason for hiding this comment

brianmcarey commented Sep 7, 2023

brianmcarey commented Sep 8, 2023

brianmcarey commented Sep 11, 2023

brianmcarey commented Sep 12, 2023

brianmcarey commented Sep 12, 2023

brianmcarey commented Sep 13, 2023

kubevirt-bot commented Sep 13, 2023

brianmcarey left a comment

Choose a reason for hiding this comment

kubevirt-bot commented Sep 13, 2023

brianmcarey commented Sep 6, 2023 •

edited

Loading

akalenyu commented Sep 6, 2023 •

edited

Loading