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

Added etcdutl in docs and modified Quickstart page to have a link to etcdutl #478

Closed
wants to merge 3 commits into from

Conversation

Somoshree
Copy link
Contributor

@Somoshree Somoshree commented Sep 15, 2021

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

I think that it is best for @nate-double-u, @spzala and you to agree to a strategy for the Demo page first (#402) so that we can find a suitable place for the new etcdutl material.

Regardless, the new material reads more like a man page than a tutorial. Maybe that will need to be reworked. Or maybe we need a Commands section to contain material like that which you have written.

@jberkus
Copy link
Contributor

jberkus commented Nov 19, 2021

@chalin

So the strategy has been to pull off the pieces of the Demo page that actually represent otherwise undocumented topics, and give them their own headers. Given that, are you OK with this page? Even if it's man-page-like, it's more documentation on etcdutl than we have in our current docs.

@chalin
Copy link
Contributor

chalin commented Nov 19, 2021

Hi @jberkus: I'm ok with the content, but it would be inappropriate to entitle it "Getting started ...". IMHO, this page should instead be under, say, op-guide/ -- such as /docs/v3.5/op-guide/cmds/etcdutl or just /docs/v3.5/op-guide/etcdutl.

@nate-double-u
Copy link
Contributor

Closing as stale.

@nate-double-u
Copy link
Contributor

Please feel free to reopen if you have time to work on this again @Somoshree, we appreciate your efforts!

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.

Add etcdutl documentation
4 participants