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

jewel: mon: osd crush set crushmap need sanity check #16144

Merged
merged 2 commits into from Sep 6, 2017

Conversation

Projects
None yet
3 participants
@smithfarm
Contributor

smithfarm commented Jul 5, 2017

@smithfarm smithfarm self-assigned this Jul 5, 2017

@smithfarm smithfarm added this to the jewel milestone Jul 5, 2017

@smithfarm smithfarm requested a review from Jul 5, 2017

@ghost

ghost approved these changes Jul 6, 2017

songbaisen and others added some commits Mar 31, 2016

crush: when take place the crush map should consider the rule is in …
…used

 Signed-off-by: song baisen song.baisen@zte.com.cn

(cherry picked from commit 09bf6f2)
mon: osd crush set crushmap need sanity check
The sanity check verifying the new crushmap does not remove crush rules
that are in use is not exclusive to ceph setcrushmap.

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

Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit ed76045)

@smithfarm smithfarm requested a review from tchaikov Sep 3, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 3, 2017

Contributor

@tchaikov This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

Can you review?

Contributor

smithfarm commented Sep 3, 2017

@tchaikov This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

Can you review?

@smithfarm smithfarm requested a review from jdurgin Sep 5, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 5, 2017

Contributor

@jdurgin Can you review?

Contributor

smithfarm commented Sep 5, 2017

@jdurgin Can you review?

@jdurgin

jdurgin approved these changes Sep 6, 2017

@smithfarm smithfarm merged commit 44cd2b1 into ceph:jewel Sep 6, 2017

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-20364-jewel branch Sep 6, 2017

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