Skip to content
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

Add kubectl overview page #14177

Merged

Conversation

hw-qiaolei
Copy link
Contributor

This is an effort try to fix: #11814

@bgrant0607 @hurf

@k8s-bot
Copy link

k8s-bot commented Sep 18, 2015

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain to ixdy.

@bgrant0607
Copy link
Member

Thanks!

cc @kubernetes/kubectl

@bgrant0607
Copy link
Member

@RichieEscarez Please suggest where links to this should be added, or existing links to the generated docs should be changed to point here.

@k8s-github-robot k8s-github-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 18, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/L


## Common Operations

For explanation, here I gave some mostly often used `kubectl` command examples. Please replace sample names with actually values if you would like to try these commands.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/actually/actual

@j3ffml
Copy link
Contributor

j3ffml commented Sep 18, 2015

This is a good intro page. My only concern is that it will not stay up to date unless it's autogenerated. A good solution would be to modify gen_kubectl_docs.go to generate this doc, if you're interested in doing that. In particular the examples, operations, and resource types should be retrieved from the built kubectl. I'm also fine with submitting this as is (once appropriate links to/from existing kubectl docs are updated) and putting a note in #14177 to autogenerate this file.

@janetkuo
Copy link
Member

Agree on auto-generate this doc. If we're submitting this as is please add kubectl edit too. https://github.com/kubernetes/kubernetes/blob/master/docs/user-guide/kubectl/kubectl_edit.md

@hw-qiaolei
Copy link
Contributor Author

@jlowdermilk thank you for reviewing this and giving these good suggestions. Yes this page is manually written and will be out of date if there is a change of kubectl. I have added the following line in this page:TODO: auto-generate this file to stay up with kubectl changes. Please see #14177.

Next step I will look into how to auto-generate it to stay up with kubectl's changes.

But please be noted that the examples in this page is expected to be oftenly used, small and stable, more user-friendly, rather than complete and exactly the same with kubectl, I rewrote these examples(not all of the commands have an example, not all of the examples given by a certain command are quoted, and I changed some variables, explanations etc).

@hw-qiaolei
Copy link
Contributor Author

@janetkuo thank you for the comment. I have added kubectl edit manually and will look into how to auto-generate this doc.

@hw-qiaolei
Copy link
Contributor Author

cc @alfred-huangjian

@hw-qiaolei hw-qiaolei force-pushed the add_kubectl_overview_page branch 2 times, most recently from f455135 to 2ac0e15 Compare September 19, 2015 02:03
@RichieEscarez
Copy link
Contributor

Sorry for the delay. Starting my review now.

@RichieEscarez RichieEscarez added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 29, 2015
@RichieEscarez
Copy link
Contributor

After a discussion with @jlowdermilk, im going to LGTM this to make it live and then open a separate PR for my edits.

@k8s-github-robot
Copy link

@k8s-bot ok to test

pr builder appears to be missing, activating due to 'lgtm' label.

@k8s-bot
Copy link

k8s-bot commented Sep 29, 2015

Unit, integration and GCE e2e test build/test passed for commit 2ac0e15.

brendandburns added a commit that referenced this pull request Sep 30, 2015
@brendandburns brendandburns merged commit 7ef257d into kubernetes:master Sep 30, 2015
@alfred-huangjian alfred-huangjian deleted the add_kubectl_overview_page branch September 30, 2015 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubectl lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a kubectl overview page for the user guide
9 participants