Skip to content

reef: mgr: process map before notifying clients#57065

Merged
SrinivasaBharath merged 1 commit intoceph:reeffrom
batrick:wip-65623-reef
Mar 21, 2025
Merged

reef: mgr: process map before notifying clients#57065
SrinivasaBharath merged 1 commit intoceph:reeffrom
batrick:wip-65623-reef

Conversation

@batrick
Copy link
Member

@batrick batrick commented Apr 23, 2024

backport tracker: https://tracker.ceph.com/issues/65623


backport of #56047
parent tracker: https://tracker.ceph.com/issues/64799

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

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.

Fixes: https://tracker.ceph.com/issues/64799
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 190f379)
@batrick batrick requested a review from a team as a code owner April 23, 2024 18:28
@batrick batrick added this to the reef milestone Apr 23, 2024
@batrick batrick added the core label Apr 23, 2024
@github-actions github-actions bot added the mgr label Apr 23, 2024
@batrick batrick marked this pull request as draft April 23, 2024 23:49
@batrick
Copy link
Member Author

batrick commented Apr 23, 2024

@github-actions github-actions bot added the stale label Nov 26, 2024
@batrick batrick removed the stale label Nov 26, 2024
@ceph ceph deleted a comment from github-actions bot Dec 30, 2024
@batrick
Copy link
Member Author

batrick commented Dec 30, 2024

https://tracker.ceph.com/issues/64799#note-2

It's been 9 months. Let's proceed. cc @rzarzynski

@batrick batrick marked this pull request as ready for review December 30, 2024 19:42
@mchangir
Copy link
Contributor

This PR is under test in https://tracker.ceph.com/issues/69881.

@mchangir
Copy link
Contributor

This PR is under test in https://tracker.ceph.com/issues/70178.

@shraddhaag
Copy link
Contributor

@SrinivasaBharath SrinivasaBharath merged commit f70d108 into ceph:reef Mar 21, 2025
@idryomov idryomov modified the milestones: reef, v18.2.6 Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants