Project

General

Profile

Actions
Copy link

Bug #14126

closed

librbd: return error if we are unable to remove the previously created selfmanaged_snap.

Added by xie xingguo over 8 years ago. Updated over 8 years ago.

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;
}

Actions
Copy link
 #2

Updated by Nathan Cutler over 8 years ago

  • Project changed from Ceph to rbd
Actions
Copy link
 #3

Updated by Nathan Cutler over 8 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #4

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
Copy link

Also available in: Atom PDF