Skip to content

crush: set_choose_tries = 100 for erasure code rulesets#4031

Merged
liewegas merged 1 commit intogiantfrom
unknown repository
Mar 17, 2015
Merged

crush: set_choose_tries = 100 for erasure code rulesets#4031
liewegas merged 1 commit intogiantfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Mar 17, 2015

It is common for people to try to map 9 OSDs out of a 9 OSDs total ceph
cluster. The default tries (50) will frequently lead to bad mappings for
this use case. Changing it to 100 makes no significant CPU performance
difference, as tested manually by running crushtool on one million
mappings.

http://tracker.ceph.com/issues/10353 Fixes: #10353

Signed-off-by: Loic Dachary ldachary@redhat.com
(cherry picked from commit 2f87ac8)

It is common for people to try to map 9 OSDs out of a 9 OSDs total ceph
cluster. The default tries (50) will frequently lead to bad mappings for
this use case. Changing it to 100 makes no significant CPU performance
difference, as tested manually by running crushtool on one million
mappings.

http://tracker.ceph.com/issues/10353 Fixes: #10353

Signed-off-by: Loic Dachary <ldachary@redhat.com>
(cherry picked from commit 2f87ac8)
@ghost ghost added bug-fix core labels Mar 17, 2015
@ghost ghost self-assigned this Mar 17, 2015
@ghost ghost added this to the giant milestone Mar 17, 2015
@loic-bot
Copy link

SUCCESS: the output of run-make-check.sh on centos-7 for 048fcc8 is http://paste2.org/FPm43XvP

:octocat: Sent from GH.

liewegas added a commit that referenced this pull request Mar 17, 2015
crush: set_choose_tries = 100 for erasure code rulesets
@liewegas liewegas merged commit 9094185 into ceph:giant Mar 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants