-
Notifications
You must be signed in to change notification settings - Fork 530
make create-clusters.sh work based on kind document #1323
Conversation
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 a nit, otherwise lgtm.
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.
Can we please update the example to use a real tag, and I'd really like it if we could take this opportunity to upgrade to latest kind images.
and update KIND_TAG to the latest version
I make updating tag version together with the fix in 1 commit because updating tag version is simple and easy to understand. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jimmidyson, xunpan 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 this PR does / why we need it:
The script should work based on documents.
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 #1322
Special notes for your reviewer: