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

ceph.conf #257

Open
swiftgist opened this issue May 16, 2017 · 2 comments
Open

ceph.conf #257

swiftgist opened this issue May 16, 2017 · 2 comments

Comments

@swiftgist
Copy link
Contributor

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
Copy link
Contributor

jan--f commented May 17, 2017

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
Copy link
Contributor Author

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.

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

No branches or pull requests

2 participants