-
Notifications
You must be signed in to change notification settings - Fork 326
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
bump kind and kubectl versions and update k8s support #4244
Conversation
bump kind and kubectl versions, and update k8s versions tested
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.
One more edit we should make, otherwise LGTM!
The acceptance tests that run for this PR should use the updated values automatically, we can observe them being updated here for the latest commit (note the kindest/node
image version isn't updated yet): https://github.com/hashicorp/consul-k8s-workflows/actions/runs/10360967522/job/28680500788#step:1:141
Co-authored-by: Michael Zalimeni <michael.zalimeni@hashicorp.com>
@zalimeni thank you ; committed your suggestion. |
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.
Thanks @NicoletaPopoviciu !
Sorry @NicoletaPopoviciu , overlooked this while reviewing. I think yes to this change, then we'll want some targeted changes similar to it, but slightly different, for |
bump kind and kubectl versions, and update k8s versions tested
Changes proposed in this PR
How I've tested this PR
How I expect reviewers to test this PR
Checklist