Project

General

Profile

Bug #24000

mon: snap delete on deleted pool returns 0 without proper payload

Added by Jason Dillaman over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Correctness/Safety
Target version:
-
Start date:
05/03/2018
Due date:
% Done:

0%

Source:
Tags:
Backport:
mimic
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Monitor
Pull request ID:

Description

It can lead to an abort in the client application since an empty reply w/o an error code is constructed in the monitor and the decode of the reply isn't wrapped with a try/catch block.

2018-05-03 18:58:44.207 7f9a28b11700 10 mon.a@0(leader).osd e104 attempt to operate on non-existent pool id 2

http://pulpito.ceph.com/jdillaman-2018-05-03_14:25:40-rbd-wip-rbd-mirror-policy-distro-basic-smithi/2471988


Related issues

Copied to RADOS - Backport #24103: mimic: mon: snap delete on deleted pool returns 0 without proper payload Resolved

History

#1 Updated by Jason Dillaman over 1 year ago

  • Status changed from In Progress to Need Review

#2 Updated by Greg Farnum over 1 year ago

  • Subject changed from Potential race with creating self-managed snapshot and pool deletion to mon: snap delete on deleted pool returns 0 without proper payload
  • Category set to Correctness/Safety
  • Status changed from Need Review to Verified
  • Component(RADOS) Monitor added

Jason put a client-side handler in, but we should change the monitor as well so that we don't break older clients (or tell lies!).

#3 Updated by Greg Farnum over 1 year ago

  • Assignee deleted (Jason Dillaman)

#4 Updated by Josh Durgin over 1 year ago

  • Status changed from Verified to Testing

#5 Updated by Kefu Chai over 1 year ago

  • Status changed from Testing to Pending Backport
  • Backport set to mimic

#6 Updated by Kefu Chai over 1 year ago

  • Copied to Backport #24103: mimic: mon: snap delete on deleted pool returns 0 without proper payload added

#7 Updated by Nathan Cutler over 1 year ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF