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

mon: osd crush set crushmap need sanity check #14029

Merged
merged 1 commit into from Apr 7, 2017

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Mar 18, 2017

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

@ghost ghost added bug fix mon labels Mar 18, 2017

@ghost ghost requested a review from liewegas Mar 18, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented Mar 18, 2017

Good catch! Probably want a tracker ticket to make sure this fix gets backported?

@liewegas liewegas added the needs-qa label Mar 18, 2017

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>
@ghost

This comment has been minimized.

ghost commented Mar 18, 2017

@liewegas right :-) Updated & repushed

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Apr 7, 2017

@tchaikov tchaikov merged commit 2950ad9 into ceph:master Apr 7, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment