Skip to content

hack: add --help to create-kind-cluster.sh and fix a bashism - #825

Closed
Yuan Gao (ygao-g) wants to merge 1 commit into
agent-substrate:mainfrom
ygao-g:kind-cluster-usage
Closed

hack: add --help to create-kind-cluster.sh and fix a bashism#825
Yuan Gao (ygao-g) wants to merge 1 commit into
agent-substrate:mainfrom
ygao-g:kind-cluster-usage

Conversation

@ygao-g

Copy link
Copy Markdown
Contributor

hack/create-kind-cluster.sh is configured entirely through the environment, which was only discoverable by reading it. Add a usage block that names the variables and exits before touching Docker, so --help works even where Docker is not running.

Also switch the registry's running check from == to =, the POSIX test operator the rest of the script already uses.

Split out of a larger IPv6 branch so the address-family change that follows is purely about address families.

Testing

  • hack/create-kind-cluster.sh --help
  • hack/verify/boilerplate.sh

No behaviour change to cluster creation.

The script is configured entirely through the environment, which was only
discoverable by reading it. Add a usage block that names the variables and
exits before touching Docker.

Also switch the registry's running check from `==` to `=`; the latter is the
POSIX test operator and the rest of the script already uses it.
@BenTheElder

Copy link
Copy Markdown
Collaborator

Also switch the registry's running check from == to =, the POSIX test operator the rest of the script already uses.

it's explicitly a bash script, let's use bashism?

@ygao-g

Copy link
Copy Markdown
Contributor Author

Fair — dropped it, the line is untouched now. Folded into #827, which is one commit and carries the --help block; closing this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants