Skip to content

reef: OSDMonitor: exclude destroyed OSDs from "ceph node ls" output#62326

Merged
Naveenaidu merged 1 commit intoceph:reeffrom
NitzanMordhai:wip-70495-reef
Apr 10, 2025
Merged

reef: OSDMonitor: exclude destroyed OSDs from "ceph node ls" output#62326
Naveenaidu merged 1 commit intoceph:reeffrom
NitzanMordhai:wip-70495-reef

Conversation

@NitzanMordhai
Copy link
Contributor

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


backport of #62243
parent tracker: https://tracker.ceph.com/issues/70400

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

After destroying an OSD, the command "ceph node ls" still listed the destroyed OSD,
causing confusion. This fix ensures that the status of the OSD is checked before
adding it to the output, aligning its behavior with "ceph osd tree," which already
displays the correct status.

Fixes: https://tracker.ceph.com/issues/70400
Signed-off-by: Nitzan Mordechai <nmordech@redhat.com>
(cherry picked from commit 97c43c7)
@NitzanMordhai NitzanMordhai requested a review from a team as a code owner March 16, 2025 09:42
@NitzanMordhai NitzanMordhai added this to the reef milestone Mar 16, 2025
@github-actions github-actions bot added the mon label Mar 16, 2025
@Naveenaidu
Copy link
Contributor

RADOS Approved: https://tracker.ceph.com/issues/70518#note-13

@yuriw
Copy link
Contributor

yuriw commented Apr 9, 2025

@Naveenaidu pls approve and merge
ref: https://tracker.ceph.com/issues/70518

@Naveenaidu Naveenaidu merged commit ed2584b into ceph:reef Apr 10, 2025
10 checks passed
@FredNass
Copy link
Contributor

FredNass commented Aug 8, 2025

@yuriw That fix doesn't appear in the release notes for 18.2.7. Was it an oversight in the release notes, or was the fix not actually included in 18.2.7?

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.

4 participants