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

doc: add descriptions for mon/mgr options #15032

Merged
merged 1 commit into from May 17, 2017

Conversation

Projects
None yet
3 participants
@tchaikov
Contributor

tchaikov commented May 10, 2017

Signed-off-by: Kefu Chai kchai@redhat.com

@tchaikov tchaikov requested a review from johnwilkins May 10, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 10, 2017

@johnwilkins it's still a work-in-progress. will finish it tomorrow.

@tchaikov tchaikov changed the title from [WIP] doc: add descriptions for mon/mgr options to doc: add descriptions for mon/mgr options May 11, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 11, 2017

@johnwilkins i just finished updating the missing doc for mon options.

@tchaikov tchaikov requested a review from jecluis May 12, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 12, 2017

@jecluis could you help proof-read the descriptions for the mon options?

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented May 12, 2017

Nice Explanation! 👍

doc: add descriptions for mon/mgr options
Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 16, 2017

@jecluis could you take a quick look at this PR? just in case i am terribly wrong about some of the options.

@johnwilkins johnwilkins merged commit 3b943b4 into ceph:master May 17, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@tchaikov tchaikov deleted the tchaikov:wip-doc-opts branch May 17, 2017

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