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

crush, mon: make jewel the lower bound for client/crush compat for new clusters #15370

Merged
merged 4 commits into from Jun 1, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented May 30, 2017

No description provided.

@ceph-jenkins

This comment has been minimized.

Collaborator

ceph-jenkins commented May 31, 2017

all commits in this PR are signed

@liewegas

This comment has been minimized.

Member

liewegas commented May 31, 2017

retest this please

@liewegas liewegas removed the needs-review label May 31, 2017

liewegas added some commits May 30, 2017

crush: set default tunables to jewel
For new clusters.

This is the last "disruptive" tunable change we've made (where adjusting/
fixing it requires lots of data movement).  For new clusters a lower bound
of jewel seems pretty reasonable, and this has been upstream in Linux
for a while now.

Signed-off-by: Sage Weil <sage@redhat.com>
mon: set mon_osd_initial_require_min_compat_client = jewel
IMO a jewel lower bound is reasonable for new clusters.

Signed-off-by: Sage Weil <sage@redhat.com>
test/cli/crushtool: adjust tests
Specify chooseleaf_stable for --build tests to match prior output.

Signed-off-by: Sage Weil <sage@redhat.com>
test/cli/osdmaptool/: fix tests
CRUSH tunables now default to jewel; outputs vary and
upmap results are a bit different.

Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas merged commit 037d6a7 into ceph:master Jun 1, 2017

2 of 3 checks passed

default Build started sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details

@liewegas liewegas deleted the liewegas:wip-jewel-tunables branch Jun 1, 2017

tchaikov added a commit to tchaikov/ceph that referenced this pull request Jun 1, 2017

test/crush: adjust for new default default tunnable
see also ceph#15370

Signed-off-by: Kefu Chai <kchai@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment