Project

General

Profile

Actions

Bug #64799

open

mgr: update cluster state for new maps from the mons before notifying modules

Added by Patrick Donnelly about 2 months ago. Updated 10 days ago.

Status:
Pending Backport
Priority:
Normal
Category:
ceph-mgr
Target version:
% Done:

0%

Source:
Development
Tags:
backport_processed
Backport:
squid,reef,quincy
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://github.com/ceph/ceph/blob/639d182732644edc5c413562ebc904ab5b953303/src/mgr/Mgr.cc#L590-L597

Oddly the maps are processed after notifying the modules. I'm not sure if this actually can cause modules to read from the old maps (since finisher contexts are queued) but it seemed odd.


Related issues 3 (3 open0 closed)

Copied to mgr - Backport #65621: quincy: mgr: update cluster state for new maps from the mons before notifying modulesIn ProgressPatrick DonnellyActions
Copied to mgr - Backport #65622: squid: mgr: update cluster state for new maps from the mons before notifying modulesIn ProgressPatrick DonnellyActions
Copied to mgr - Backport #65623: reef: mgr: update cluster state for new maps from the mons before notifying modulesIn ProgressPatrick DonnellyActions
Actions #1

Updated by Patrick Donnelly about 2 months ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 56047
Actions #2

Updated by Radoslaw Zarzynski 15 days ago

Per let's not hurry up with backporting this chnage. IMHO it deserves some baking in `main`.:

let's not hurry up with backporting this chnage. IMHO it deserves some baking in `main`.

Actions #3

Updated by Patrick Donnelly 10 days ago

  • Status changed from Fix Under Review to Pending Backport

I'll sit on the backports for a while.

Actions #4

Updated by Backport Bot 10 days ago

  • Copied to Backport #65621: quincy: mgr: update cluster state for new maps from the mons before notifying modules added
Actions #5

Updated by Backport Bot 10 days ago

  • Copied to Backport #65622: squid: mgr: update cluster state for new maps from the mons before notifying modules added
Actions #6

Updated by Backport Bot 10 days ago

  • Copied to Backport #65623: reef: mgr: update cluster state for new maps from the mons before notifying modules added
Actions #7

Updated by Backport Bot 10 days ago

  • Tags set to backport_processed
Actions

Also available in: Atom PDF