Skip to content

squid: OSDMonitor: exclude destroyed OSDs from "ceph node ls" output#62327

Merged
SrinivasaBharath merged 1 commit intoceph:squidfrom
NitzanMordhai:wip-70496-squid
Mar 25, 2025
Merged

squid: OSDMonitor: exclude destroyed OSDs from "ceph node ls" output#62327
SrinivasaBharath merged 1 commit intoceph:squidfrom
NitzanMordhai:wip-70496-squid

Conversation

@NitzanMordhai
Copy link
Contributor

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


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
Copy link
Contributor Author

@NitzanMordhai
Copy link
Contributor Author

@rzarzynski please approve

@SrinivasaBharath
Copy link
Contributor

jenkins test make check

@SrinivasaBharath SrinivasaBharath merged commit 1b83806 into ceph:squid Mar 25, 2025
10 checks passed
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.

3 participants