jewel: fs: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup #10997

Merged
merged 1 commit into from Oct 11, 2016

Projects

None yet

2 participants

@batrick
Contributor
batrick commented Sep 6, 2016 edited
@batrick batrick added this to the jewel milestone Sep 6, 2016
@dachary
Member
dachary commented Oct 10, 2016

jenkins test this please (the logs are gone)

@dachary dachary changed the title from mds: remove max_mds config option to jewel: mds: remove max_mds config option Oct 10, 2016
@dachary
Member
dachary commented Oct 10, 2016

@batrick could you please git cherry-pick -x instead so that it has a well formed cherry picked from... line ? It's not mandatory but will help scripts that expect it. Thanks :-)

@dachary dachary self-assigned this Oct 10, 2016
@dachary dachary changed the title from jewel: mds: remove max_mds config option to jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup Oct 10, 2016
@batrick batrick mds: remove max_mds config option
It is now required that all changes to max_mds use the run-time `ceph fs
set max_mds` command. The rationale for this change is that it is
confusing to have a configuration for max_mds which is only observed at
file system creation.

Fixes: http://tracker.ceph.com/issues/17105

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 52f4e54)
c2d4239
@batrick
Contributor
batrick commented Oct 10, 2016

@dachary done.

@dachary
Member
dachary commented Oct 10, 2016

jenkins test this please (bluefs core)

@dachary
Member
dachary commented Oct 10, 2016

@batrick perfect, thanks !

@dachary
Member
dachary commented Oct 11, 2016

jenkins test this please (bluefs core)

@dachary dachary removed the needs-qa label Oct 11, 2016
@dachary dachary merged commit 59f0f4c into ceph:jewel Oct 11, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@batrick batrick deleted the batrick:i17105-backport branch Oct 11, 2016
@theanalyst theanalyst changed the title from jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup to "jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup" Nov 17, 2016
@theanalyst theanalyst changed the title from "jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup" to jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup Nov 17, 2016
@theanalyst theanalyst changed the title from jewel: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup to jewel: fs: multimds: allow_multimds not required when max_mds is set in ceph.conf at startup Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment