Actions
Bug #2345
closedmon: users can create both pool snapshots and self-managed snapshots on a single pool
% Done:
0%
Source:
Development
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
When the OSD gets the map it breaks, but the Monitor will happily set both. There should be guard against that.
(See http://marc.info/?l=ceph-devel&m=133534478717808&w=2)
Updated by Greg Farnum almost 12 years ago
- Status changed from New to In Progress
Ah, there are Monitor checks for this but they aren't quite right: they look at the contents of snaps and removed_snaps, but don't consider the possibility that these things previously existed but have been erased.
I'm simplifying this by making use of the is_pool_snaps_mode() check (and adding a similar is_unmanaged_snaps_mode()).
Updated by Greg Farnum almost 12 years ago
- Status changed from In Progress to Fix Under Review
Pushed a fix (and tested it using rados snap create and rbd snap create) to wip-2345-snaps.
Updated by Sage Weil almost 12 years ago
- Status changed from Fix Under Review to Resolved
Actions