Skip to content
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: take crashes due to invalid arg #4769

Merged
1 commit merged into from Sep 4, 2015

Conversation

Projects
None yet
3 participants
@smithfarm
Copy link
Contributor

smithfarm commented May 26, 2015

mon: prevent bucket deletion when referenced by a rule
If a rule references a bucket with 'take', prevent deletion.

Fixes: #11602
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 3d591af)

@smithfarm smithfarm added this to the firefly milestone May 26, 2015

@smithfarm smithfarm self-assigned this May 28, 2015

@ghost

This comment has been minimized.

Copy link

ghost commented Jul 21, 2015

the rados/monthrash/workloads/rados_mon_workunits.yaml runs the test that was added to verify this commit and it is part of the http://pulpito.ceph.com/loic-2015-07-09_21:09:04-rados-firefly-backports---basic-multi run that passed. @liewegas do you think this backport can be merged in firefly ?

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

smithfarm commented Sep 4, 2015

@athanatos This is a backport of 6b982e4 which you merged. It has passed a rados suite as detailed by http://tracker.ceph.com/issues/11644#rados . Do you think it's OK to merge?

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

smithfarm commented Sep 4, 2015

@dachary ?

ghost pushed a commit that referenced this pull request Sep 4, 2015

Loic Dachary
Merge pull request #4769 from SUSE/wip-11741-firefly
crush: take crashes due to invalid arg

Reviewed-by: Loic Dachary <ldachary@redhat.com>

@ghost ghost merged commit 49d416e into ceph:firefly Sep 4, 2015

@smithfarm smithfarm deleted the SUSE:wip-11741-firefly branch Sep 4, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.