-
Notifications
You must be signed in to change notification settings - Fork 519
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
Remove kubernetes 1.20 variants #2421
Conversation
Marking this draft until we're abit closer to the November timeline (when EKS support for k8s 1.20 will drop per the issue noted above) |
3c142b2
to
d7c1ff5
Compare
Ready for review now and will target the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should delete models/src/aws-k8s-1.20
like in https://github.com/bottlerocket-os/bottlerocket/pull/2316/files#.
d7c1ff5
to
72da26d
Compare
Force pushed to also remove |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure the best way to make sure nothing is missed when reviewing this type of PR. I did compare it to #2316 though and didn't surface any issues.
- Remove aws-k8s-1.20 variant - Remove vmware-k8s-1.20 variant Signed-off-by: John McBride <[email protected]>
72da26d
to
e83f600
Compare
Force pushed to address Ben's comment: Added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issue number:
Closes #2400
Reference PR: #2316
Description of changes:
Testing done:
cargo make check && cargo make build
👍🏼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.