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
Upgrade to use latest Kind version v0.12.0 #3293
Upgrade to use latest Kind version v0.12.0 #3293
Conversation
@shivi28: This issue is currently awaiting triage. If CAPA/CAPI contributors determines this is a relevant issue, they will accept it by applying the The 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/test-infra repository. |
/test pull-cluster-api-provider-aws-e2e |
/test pull-cluster-api-provider-aws-e2e-eks |
/lgtm |
I don't see Shouldn't it be referenced from ci scripts? |
Nice observation @pydctw. Thanks for bringing this. |
0a3f4e0
to
17e827b
Compare
/test pull-cluster-api-provider-aws-e2e-eks |
Hey @shivi28, thanks for the PR 🎉 |
IMO all the version bump related to same library/tool should go in same PR, that's why closed the other PR so that we can merge both together in one. For example, we closed this dependabot PR such that whole cluster api bump goes in one PR. |
+1 to keeping related changes to the same PR. I'm not sure if we have permission to add commits to existing PRs in this repo; if we do, then adding commits to the dependabot PR #3305 is an option. |
Atleast when I tried, I have no permission to do so. I think maintainers have that rights to amend to dependabot PR but I m not super sure. |
17e827b
to
14fde5a
Compare
/test pull-cluster-api-provider-aws-e2e |
/lgtm |
Looks good to me. Thanks for also adding ensure-kubectl.sh to CI scripts. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sedefsavas The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/area release
What this PR does / why we need it:
This updates to make use of latest kind v0.12.0 version
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Checklist:
Release note: