-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Issue 3535: clusterctl get kubeconfig docs added #3644
Issue 3535: clusterctl get kubeconfig docs added #3644
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.
Thanks for this PR!
When I clicked on the link in the quickstart, the page 404'd for me. So not sure what's going on there.
Also add a reference to the commands page and the index page.
@@ -0,0 +1,18 @@ | |||
# clusterctl get kubeconfig | |||
|
|||
The `clusterctl get` command prints the kubeconfig of an existing workload cluster into stdout. |
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.
There could be more sub-commands in the future under the get
verb.
The `clusterctl get` command prints the kubeconfig of an existing workload cluster into stdout. | |
This command prints the kubeconfig of an existing workload cluster into stdout. |
# clusterctl get kubeconfig | ||
|
||
The `clusterctl get` command prints the kubeconfig of an existing workload cluster into stdout. | ||
This funcitonality is available in clusterctl version 0.3.9 or newer. |
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 funcitonality is available in clusterctl version 0.3.9 or newer. | |
This functionality is available in clusterctl v0.3.9 or newer. |
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.
Thankyou for the PR!
Adding to Warren's point, the command should also be referenced in the overview page.
The `clusterctl get` command prints the kubeconfig of an existing workload cluster into stdout. | ||
This funcitonality is available in clusterctl version 0.3.9 or newer. | ||
|
||
## Usage: |
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.
## Usage: |
For sake of consistency with others command's doc, we should drop Usage.
Also, I suggest to drop the list of parameters; instead, you can provide an example of calling the command with the --namespace
flag
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.
ack
10dfd2d
to
90b8cf3
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.
Lgtm except for the one nit.
docs/book/src/SUMMARY.md
Outdated
@@ -20,6 +20,7 @@ | |||
- [init](clusterctl/commands/init.md) | |||
- [config cluster](clusterctl/commands/config-cluster.md) | |||
- [generate yaml](clusterctl/commands/generate-yaml.md) | |||
- [clusterctl get kubeconfig](clusterctl/commands/get-kubeconfig.md) |
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.
nit: For consistency
- [clusterctl get kubeconfig](clusterctl/commands/get-kubeconfig.md) | |
- [get kubeconfig](clusterctl/commands/get-kubeconfig.md) |
LGTM |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini 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 |
90b8cf3
to
7e04a45
Compare
/lgtm |
📖
What this PR does / why we need it: Adds documentation clusterctl get kubeconfig command
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 #3535
/cc @prankul88 @fabriziopandini @wfernandes