populate the instructions on installing kubectl in getting-started-guides #11276
Conversation
|
GCE e2e build/test passed for commit e981a39d3ad946253cca436a5979bf2c69edd3d1. |
|
I will address this comment in this PR. cc @bgrant0607 Update: Pushed |
caesarxuchao
force-pushed
the
doc1
branch
2 times, most recently
from
62d4458 to
fa64fbc
Compare
|
GCE e2e build/test failed for commit 62d4458a463ad077b0e06266e52819ec35e0a494. |
|
cc @erictune |
|
GCE e2e build/test failed for commit fa64fbcaf27458561252266be56ea80361a4c1d3. |
| @@ -38,7 +38,7 @@ no security tokens, no basic auth). For demonstration purposes only. | |||
|
|
|||
| * [aws CLI](http://aws.amazon.com/cli) | |||
| * [CoreOS image for AWS](https://coreos.com/docs/running-coreos/cloud-providers/ec2/) | |||
| * [kubectl CLI](aws/kubectl.md) | |||
| * [kubectl CLI](aws/kubectl.md) ([installation](https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/getting-started-guides/aws.md#command-line-administration-tool-kubectl)) | |||
|
@jlowdermilk @deads2k PTAL |
|
The user guide TOC is in docs/user-guide/README.md, but I would link directly to docs/user-guide/prereqs.md. |
| export PATH=<path/to/kubernetes-directory>/platforms/linux/amd64:$PATH | ||
| ``` | ||
|
|
||
| An alternate method, useful if you're building from source and do not want to configure the PATH, is using ./cluster/kubectl.sh instead of kubectl. That script will auto-detect the location of kubectl and proxy commands to it (eg., ./cluster/kubectl.sh cluster-info). |
There was a problem hiding this comment.
Would prefer we not direct users to this method. We want to stop using cluster/kubectl.sh as a command proxy and instead have it find the binary location and export PATH=...
There was a problem hiding this comment.
Bonus points if someone can better organize and cross-refernece the kubectl installation and kubeconfig docs, including this one, the text in the getting-started guides and examples (e.g., docs/user-guide/walkthrough), docs/user-guide/accessing-the-cluster.md, docs/user-guide/sharing-clusters.md, ...
There was a problem hiding this comment.
Agree we shouldn't mention kubectl.sh, btw. See #8817
|
I've pushed a commit to address the comments. @bgrant0607 I'm not sure what this comment is about. |
|
LGTM. Please squash. |
bgrant0607
added
lgtm
|
Squashed. Thanks. |
|
GCE e2e build/test passed for commit a9f979bc739578161317240f2b28d864421a4a58. |
|
LGTM |
|
GCE e2e build/test passed for commit 10d76a5. |
populate the instructions on installing kubectl in getting-started-guides
Fixes #9632 and some nits in getting-started-guides
Regarding this comment, because there is no TOC page for docs/user-guide, so I followed the previous comment
@nikhiljindal @janetkuo