Project

General

Profile

Bug #38899

mgr: handle_conf_change - broken locking

Added by xie xingguo over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
nautilus,mimic,luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature:

Description

The is_locked_by_me() is only meant to be safe if you know you hold the lock. It's meant to be used in an assertion.


Related issues

Copied to Ceph - Backport #38962: luminous: DaemonServer::handle_conf_change - broken locking Resolved
Copied to Ceph - Backport #38963: mimic: DaemonServer::handle_conf_change - broken locking Resolved
Copied to Ceph - Backport #38964: nautilus: DaemonServer::handle_conf_change - broken locking Resolved

History

#1 Updated by Mykola Golub over 1 year ago

  • Pull request ID set to 27144

#2 Updated by Mykola Golub over 1 year ago

  • Status changed from New to Fix Under Review

#3 Updated by xie xingguo over 1 year ago

  • Pull request ID changed from 27144 to 27184

#4 Updated by Sage Weil over 1 year ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport set to nautilus, mimic, luminous

#5 Updated by Nathan Cutler over 1 year ago

  • Copied to Backport #38962: luminous: DaemonServer::handle_conf_change - broken locking added

#6 Updated by Nathan Cutler over 1 year ago

  • Copied to Backport #38963: mimic: DaemonServer::handle_conf_change - broken locking added

#7 Updated by Nathan Cutler over 1 year ago

  • Copied to Backport #38964: nautilus: DaemonServer::handle_conf_change - broken locking added

#8 Updated by Patrick Donnelly over 1 year ago

  • Project changed from Ceph to mgr
  • Subject changed from DaemonServer::handle_conf_change - broken locking to mgr: handle_conf_change - broken locking
  • Target version set to v15.0.0
  • Start date deleted (03/23/2019)
  • Backport changed from nautilus, mimic, luminous to nautilus,mimic,luminous
  • Affected Versions deleted (v15.0.0)

#9 Updated by Nathan Cutler over 1 year ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved".

Also available in: Atom PDF