Skip to content

Commit

Permalink
Fix list-osd for the time being
Browse files Browse the repository at this point in the history
See https://tracker.ceph.com/issues/62320 - with the latest Ceph
releases ceph-volume lvm list does not work if there LVs with the
same name across multiple VGs.

With this patch this works again.

The patch can be removed, once Ceph with the following fix has
been released:

ceph/ceph#53841
  • Loading branch information
href committed Nov 21, 2023
1 parent 8798ccb commit 5e3b2a8
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion library/ceph_volume.py
Original file line number Diff line number Diff line change
Expand Up @@ -430,7 +430,9 @@ def list_osd(module, container_image):
cluster,
mounts={'/var/lib/ceph': '/var/lib/ceph:ro'})
if data:
cmd.append(data)
# Use the full path to the OSD until the following bug is patched:
# https://tracker.ceph.com/issues/62320
cmd.append('/dev/' + data)
cmd.append('--format=json')

return cmd
Expand Down

0 comments on commit 5e3b2a8

Please sign in to comment.