Update Karpenter to v1.6.2

[hakman]

Iterating on updated Karpenter setup.
Includes:

• storing a copy of the startup script that can be used later by kops-controller to generate the EC2NodeClass
• disabling LaunchTemplate generation, which can no longer be used by Karpenter.

/cc @rifelpet @justinsb @ameukam

[k8s-ci-robot]

[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 johngmyers for approval. For more information see the 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:

• OWNERS

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

[hakman]

/override pull-kops-e2e-aws-upgrade-k129-ko129-to-k130-kolatest-karpenter

[k8s-ci-robot]

@hakman: Overrode contexts on behalf of hakman: pull-kops-e2e-aws-upgrade-k129-ko129-to-k130-kolatest-karpenter

In response to this:

/override pull-kops-e2e-aws-upgrade-k129-ko129-to-k130-kolatest-karpenter

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-sigs/prow repository.

[rifelpet]

FYI I started working on a model that translates an InstanceGroup to EC2NodeClass and NodePool resources:

master...rifelpet:kops:karpenter-model

I was imagining that we'd add a new karpenter controller to kops-controller that periodically fetches the instance groups (and this new userdata object) from VFS, generates the model, and reconciles the resulting resources. It should be straight forward to add models for other cloud providers too.

[hakman]

FYI I started working on a model that translates an InstanceGroup to EC2NodeClass and NodePool resources:

master...rifelpet:kops:karpenter-model

I was imagining that we'd add a new karpenter controller to kops-controller that periodically fetches the instance groups (and this new userdata object) from VFS, generates the model, and reconciles the resulting resources. It should be straight forward to add models for other cloud providers too.

Very nice @rifelpet, I think that is exactly the missing piece. This PR adds generates a file with the user data, which is not that easy to generate separately. This way, everything needed for generating the EC2NodeClass is in the state store.

[hakman]

@rifelpet @justinsb could you take a look at this one, I think it should be ready.

[hakman]

/test pull-kops-verify-terraform

[hakman]

/override pull-kops-e2e-aws-karpenter

[k8s-ci-robot]

@hakman: Overrode contexts on behalf of hakman: pull-kops-e2e-aws-karpenter

In response to this:

/override pull-kops-e2e-aws-karpenter

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-sigs/prow repository.

[hakman]

/test pull-kops-scenario-aws-karpenter

[hakman]

/test pull-kops-scenario-aws-karpenter

[hakman]

/test pull-kops-scenario-aws-karpenter

[hakman]

/test pull-kops-scenario-aws-karpenter

[hakman]

/test pull-kops-kubernetes-e2e-ubuntu-gce-build

[hakman]

/override pull-kops-e2e-aws-upgrade-k133-ko133-to-klatest-kolatest-many-addons

[k8s-ci-robot]

@hakman: Overrode contexts on behalf of hakman: pull-kops-e2e-aws-upgrade-k133-ko133-to-klatest-kolatest-many-addons

In response to this:

/override pull-kops-e2e-aws-upgrade-k133-ko133-to-klatest-kolatest-many-addons

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-sigs/prow repository.

[hakman]

@justinsb @rifelpet @ameukam please take another look, I think it's ready to merge.
I will add more updates to the docs and release notes separately.

[ameukam]

/lgtm
/hold

If others have opinions.
I would have expected that peter's get merged in this PR before we merge it.

[ameukam]

/retest

[k8s-ci-robot]

@hakman: 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
pull-kops-kubernetes-e2e-ubuntu-gce-build	64a8e79	link	false	/test pull-kops-kubernetes-e2e-ubuntu-gce-build

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-sigs/prow repository. I understand the commands that are listed here.

[hakman]

I would have expected that peter's get merged in this PR before we merge it.

@ameukam I think we decided to merge and wait for Peter's PR another 3 weeks.