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

updated "sample.ceph.conf": #1080

Merged
merged 1 commit into from Jan 27, 2014
Merged

updated "sample.ceph.conf": #1080

merged 1 commit into from Jan 27, 2014

Conversation

onlyjob
Copy link
Contributor

@onlyjob onlyjob commented Jan 14, 2014

I've made the following changes to simplify initial cluster configuration:

  • Removed deprecated option(s).
  • Removed overrides for default paths that are "not recommended to change".
  • Added comments with keys' descriptions, types and defaults.
  • Added references to relevant on-line docs;
    sorted keys according to their respective docs sections (pages).
  • Use different types of comments for annotations ('#')
    and for keys/values (';').
  • Added many useful keys.
  • Commented some lines to make sample config "neutral". The idea is to
    let users to un-comment what they need rather than remove sample sections
    that they surely won't be using as is.

Signed-off-by: Dmitry Smirnov onlyjob@member.fsf.org

I've made the following changes to simplify initial cluster configuration:

 * Removed deprecated option(s).
 * Removed overrides for default paths that are "not recommended to change".
 * Added comments with keys' descriptions, types and defaults.
 * Added references to relevant on-line docs;
   sorted keys according to their respective docs sections (pages).
 * Use different types of comments for annotations ('#')
   and for keys/values (';').
 * Added many useful keys.
 * Commented some lines to make sample config "neutral". The idea is to
   let users to un-comment what they need rather than remove sample sections
   that they surely won't be using as is.

Signed-off-by: Dmitry Smirnov <onlyjob@member.fsf.org>
jdurgin added a commit that referenced this pull request Jan 27, 2014
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
@jdurgin jdurgin merged commit 120b3fb into ceph:master Jan 27, 2014
@jdurgin
Copy link
Member

jdurgin commented Jan 27, 2014

Thanks, that's much better than the old one!

liewegas pushed a commit that referenced this pull request Dec 14, 2016
tasks/cephfs/test_volume_client: test authentication metadata

Reviewed-by: John Spray <john.spray@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants