-
Notifications
You must be signed in to change notification settings - Fork 510
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
add k8s 1.26 variants #2771
add k8s 1.26 variants #2771
Conversation
need update after merging #2774 |
cd31647
to
0d0a951
Compare
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.
Just some questions and minor things inline. Looks good!
Blocked by EKS team that enable |
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.
This is looking great - thanks @gthao313!
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.
Built a aws-k8s-1.25 variant and deployed an ami
cargo make -e BUILDSYS_VARIANT=aws-k8s-1.26
...
cargo make -e BUILDSYS_VARIANT=aws-k8s-1.26 ami
👍🏼
|
||
%build | ||
# Build codegen programs with the host toolchain. | ||
make hack/update-codegen.sh |
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.
FYI to others since I was curious:
echo "'make generated_files' is deprecated. Please use hack/update-codegen.sh instead."
And looks like it'll be dropped in k8s 1.29
https://github.com/kubernetes/kubernetes/blob/2dae34c4a2768a8d39422be42eae2b2817a41a7c/build/root/Makefile#L311-L317
This adds a k8s-1.26 package. Existing 1.25 models are moved to be 1.26, with symlinks created for 1.25 models to point to the new locations. Finally, new variants are defined for aws-k8s-1.26, aws-k8s-1.26-nvidia, metal-k8s-1.26, and vmware-k8s-1.26.
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.
Looks good, thanks for getting the URL updated.
Issue number:
Closes #2770
Description of changes:
Testing done:
To be done:
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.