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

doc: fix "mon osd down out subtree limit" option name #7164

Merged
merged 1 commit into from Jan 20, 2016
Merged

doc: fix "mon osd down out subtree limit" option name #7164

merged 1 commit into from Jan 20, 2016

Conversation

flaf
Copy link
Contributor

@flaf flaf commented Jan 9, 2016

The name of the parameter mon osd down out subtree limit
was incorrect (with downout instead of down out).
Furthermore, the description seemed to me a little bit
ambiguous.

Signed-off-by: François Lafont francois.lafont@ac-versailles.fr

@tchaikov
Copy link
Contributor

@flaf could you update the title of your commit message:

  1. add a prefix "doc: "
  2. be more specific than "little doc improvement", something like

fix "mon osd down out subtree limit" option name

would suffice.

@tchaikov tchaikov self-assigned this Jan 20, 2016
@flaf flaf changed the title Little doc improvement doc: fix "mon osd down out subtree limit" option name Jan 20, 2016
@flaf
Copy link
Contributor Author

flaf commented Jan 20, 2016

Done.

The name of the parameter `mon osd down out subtree limit`
was incorrect (with `downout` instead of `down out`).
Furthermore, the description seemed to me a little bit
ambiguous.

Signed-off-by: François Lafont <francois.lafont@ac-versailles.fr>
@tchaikov
Copy link
Contributor

@flaf we are close. but before merge, could you remove the "merge" commit from your pr?

tchaikov added a commit that referenced this pull request Jan 20, 2016
doc: fix "mon osd down out subtree limit" option name

Reviewed-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov merged commit 9a61ac2 into ceph:master Jan 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants