kops icon indicating copy to clipboard operation
kops copied to clipboard

[DigitalOcean] use ubuntu-22-04 as the default node image

Open srikiz opened this issue 2 years ago • 7 comments

Minor update to use ubuntu-22-04 as a base image going forward. Also realized we no longer use the e2e scenarios for running tests, but have them explicitly specified in test-infra repo. So deleting this file for clarity.

FYI - @timoreimann @rifelpet

srikiz avatar Jul 19 '22 17:07 srikiz

@srikiz We are planning to move to 22.04 soon, but there is no decision yet. Would you mind waiting a little with this change?

hakman avatar Jul 19 '22 18:07 hakman

Oh okay. How about updating to ubuntu-21-10-x64 for now?

srikiz avatar Jul 19 '22 18:07 srikiz

We only use Ubuntu LTS by default. Any special reason you want to move to 22.04?

hakman avatar Jul 19 '22 18:07 hakman

@srikiz Would it be ok for you to switch back to 22.04 and I will have an answer about the timeline for the merge soon.

hakman avatar Jul 20 '22 07:07 hakman

[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 kashifsaadat for approval by writing /assign @kashifsaadat in a comment. 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:

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

k8s-ci-robot avatar Jul 22 '22 14:07 k8s-ci-robot

@srikiz Would it be ok for you to switch back to 22.04 and I will have an answer about the timeline for the merge soon.

Sure @hakman - I wasn't aware that we use Ubuntu LTS by default. I am okay to close this PR if we need to be on LTS.

srikiz avatar Jul 22 '22 14:07 srikiz

Let's see what the conclusion about 22.04 is at next kOps Office Hours.

hakman avatar Jul 22 '22 15:07 hakman

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

Test name Commit Details Required Rerun command
pull-kops-e2e-aws-karpenter 1549d3eb59f56ec2d3d7e2c0aa9ccc34e77f8760 link true /test pull-kops-e2e-aws-karpenter
pull-kops-test 1549d3eb59f56ec2d3d7e2c0aa9ccc34e77f8760 link true /test pull-kops-test

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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.

k8s-ci-robot avatar Aug 31 '22 08:08 k8s-ci-robot

Sorry, but I forgot to give you an update @srikiz. Seems that there's still no consensus about upgrading. Switching to cgroupv2 is especially risky as there isn't yet that much adoption. https://github.com/kubernetes/kops/issues/14140 will be used to track the upgrade to 22.04 in the future.

hakman avatar Sep 19 '22 13:09 hakman

Closing this until there is an agreement about https://github.com/kubernetes/kops/issues/14140.

hakman avatar Sep 24 '22 10:09 hakman