From: Wido den Hollander <wido@42on.com>
To: Sage Weil <sage@newdream.net>, Haomai Wang <haomaiwang@gmail.com>
Cc: "ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>
Subject: Re: /var/lib/ceph/*/*/config
Date: Wed, 27 Jan 2016 16:24:50 +0100 [thread overview]
Message-ID: <56A8E142.4040401@42on.com> (raw)
In-Reply-To: <alpine.DEB.2.11.1601270956260.6524@cpach.fuggernut.com>
On 27-01-16 15:59, Sage Weil wrote:
> Loic pointed out that adding a new config file that is overrides/combines
> with the global config makes our already-complicated config infrastructure
> even more so. And it turned out to be more complicated to implement than
> I'd hoped. On the other hand, I think there are valid use-cases for a
> global config, e.g.:
>
> - most config on the osd disk itself
> - /etc/ceph/ceph.conf containing crush_location and mon_host
>
> ...and with a single config I'm not sure how we'd get the datacenter
> location set properly.
>
Yes, you probably want the mon_host variable in a global config. We
might and I say *might* want to look at a DNS lookup at some time?
A lookup for 'ceph-monitor' in the current DNS search domain. But that
might be offtopic here.
> Anyway, I have two versions implemented. The first just adds
> $data_dir/config to the search pathh:
>
> https://github.com/ceph/ceph/pull/7377
>
I like the first patch, but it doesn't seem to merge the config files,
right?
If a global /etc/ceph/ceph.conf is found it doesn't search any further.
> The second adds the extra_config infrastructure. It's just the last patch
> that is different:
>
> https://github.com/ceph/ceph/pull/7378
>
The --extra-config seems weird to me. Isn't there a way where you supply
--config multiple times with multiple files?
Wido
> Comments?
> sage
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2016-01-27 15:24 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-19 15:15 /var/lib/ceph/*/*/config Sage Weil
2016-01-19 15:34 ` /var/lib/ceph/*/*/config Wido den Hollander
2016-01-19 15:48 ` /var/lib/ceph/*/*/config Allen Samuels
[not found] ` <CACJqLyaMC-UaAhTnzCeUqPCoY3HcoqLwHg3tpWW0H-+6AN6C7Q@mail.gmail.com>
2016-01-19 16:46 ` /var/lib/ceph/*/*/config Sage Weil
2016-01-19 16:55 ` /var/lib/ceph/*/*/config Haomai Wang
2016-01-27 14:59 ` /var/lib/ceph/*/*/config Sage Weil
2016-01-27 15:24 ` Wido den Hollander [this message]
2016-01-27 15:45 ` /var/lib/ceph/*/*/config Sage Weil
2016-01-27 16:03 ` /var/lib/ceph/*/*/config Wido den Hollander
2016-01-27 16:29 ` /var/lib/ceph/*/*/config Haomai Wang
2016-01-20 10:42 ` /var/lib/ceph/*/*/config John Spray
2016-01-21 6:15 ` /var/lib/ceph/*/*/config Wang, Yaguang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56A8E142.4040401@42on.com \
--to=wido@42on.com \
--cc=ceph-devel@vger.kernel.org \
--cc=haomaiwang@gmail.com \
--cc=sage@newdream.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.