search

SUSE / DeepSea

A collection of Salt files for deploying, managing and automating Ceph.
GNU General Public License v3.0
161 stars 75 forks source link

ceph.conf #257

Open swiftgist opened 7 years ago

swiftgist commented 7 years ago

PR #256 started me thinking about other issues related to ceph.conf.

mon_allow_pool_delete = false

may need to toggle in conjunction with disengage.safety runner.

The monitor hostnames/addresses are not necessary in environments that support DNS SRV records. Maybe the inclusion of mon_initial_members and mon_host should be conditional.

rbd default features = 3

is currently hardcoded. This likely needs to be re-evaulated. (Maybe this lives as an example in ceph.conf.d)

jan--f commented 7 years ago

mon_allow_pool_delete = false may need to toggle in conjunction with disengage.safety runner.

That seems like a good approach to me. This option can be set at runtime via ceph tell. I'll look into a patch for this.

The monitor hostnames/addresses are not necessary in environments that support DNS SRV records. Maybe the inclusion of mon_initial_members and mon_host should be conditional.

Doesn't ceph need this to get to a initial quorum? If so seems like a bad idea to maybe not include it...or am I misunderstanding?

swiftgist commented 7 years ago

Doesn't ceph need this to get to a initial quorum? If so seems like a bad idea to maybe not include it...or am I misunderstanding?

I made a mistake the other day and had a cluster with no mon_hosts set. Ceph naturally falls back to DNS SRV records. Some sites might be using this, but we are hardcoding the monitor addresses. I think that's still a reasonable default, but we should make that part of the ceph.conf easy to opt out.