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

Drop AWS K8s 1.23 variants #4227

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

ginglis13
Copy link
Contributor

Issue number:

Closes #4083

Description of changes:

Drop the aws-k8s-1.23 and aws-k8s-1.23-nvidia variants following the EKS support policy. The variants go EOL in EKS on October 11.

Testing done:

cargo make succeeds, cargo make -e BUILDSYS_VARIANT=aws-k8s-1.23 and cargo make -e BUILDSYS_VARIANT=aws-k8s-1.23-nvidia fail.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Drop the aws-k8s-1.23 and aws-k8s-1.23-nvidia variants following the EKS
support policy. The variants go EOL in EKS on October 11

Signed-off-by: Gavin Inglis <[email protected]>
@koooosh
Copy link
Contributor

koooosh commented Oct 7, 2024

Do we also need to remove the k8s-1.23 pkg from the core-kit?

@ginglis13
Copy link
Contributor Author

Do we also need to remove the k8s-1.23 pkg from the core-kit?

Yes that work is on my plate, but it's not a requirement for this PR

@ginglis13 ginglis13 merged commit a8d358b into bottlerocket-os:develop Oct 7, 2024
2 checks passed
@koooosh
Copy link
Contributor

koooosh commented Oct 7, 2024

Also, I see a few more hits for k8-1.23 in this repo (search link), particularly:

Should these be removed too?

@ginglis13
Copy link
Contributor Author

Also, I see a few more hits for k8-1.23 in this repo (search link), particularly:

[settings-defaults](https://github.com/bottlerocket-os/bottlerocket/blob/acfd0abee25b6061fcff30206856d36271a12d73/packages/settings-defaults/settings-defaults.spec#L72)
[settings-plugins](https://github.com/bottlerocket-os/bottlerocket/blob/acfd0abee25b6061fcff30206856d36271a12d73/packages/settings-plugins/settings-plugins.spec#L59)

Should these be removed too?

Those were only used by the k8s-1.23 variants, but good callout, I'll follow up removing those

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove aws-k8s-1.23 variants by October 2024
4 participants