Actions
Bug #14126
closedlibrbd: return error if we are unable to remove the previously created selfmanaged_snap.
Status:
Won't Fix
Priority:
Normal
Assignee:
-
Target version:
-
% Done:
0%
Source:
Community (dev)
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
See Internal.cc:
int validate_pool(IoCtx &io_ctx, CephContext *cct)
r = io_ctx.selfmanaged_snap_remove(snap_id);
if (r < 0) {
lderr(cct) << "failed to release self-managed snapshot " << snap_id
<< ": " << cpp_strerror(r) << dendl;
}
Updated by Nathan Cutler over 8 years ago
- Status changed from New to Fix Under Review
Updated by Jason Dillaman over 8 years ago
- Status changed from Fix Under Review to Won't Fix
This was an intentional choice -- no way to recover the snap id (not that you would need to anyway). Returning a failure would just result in another snap id being allocated from the monitors.
Actions