-
Notifications
You must be signed in to change notification settings - Fork 5.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ceph-volume: fix raw list with logical partition #43050
Conversation
FYI @BlaineEXE |
jenkins test ceph-volume tox |
we must be missing some tests |
This is a regression introduced by 9212420, when the host is using a logical partition then lsblk reports that partition as a child from the physical device. That logical partition is prefixed by the `└─` character. This leads the `raw list` subcommand to show the lsblk error on the stderr. ``` $ ceph-volume raw list {} stderr: lsblk: `-/dev/sda1: not a block device ``` The lsblk command output looks like: ``` $ lsblk --paths --output=NAME --noheadings /dev/sda └─/dev/sda1 /dev/sdb /dev/sdc /dev/sdd ``` Using the `--list` option with lsblk solves the issue. ``` $ lsblk --list --paths --output=NAME --noheadings /dev/sda /dev/sda1 /dev/sdb /dev/sdc /dev/sdd ``` Fixes: https://tracker.ceph.com/issues/52504 Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
Let's test we use the expected args when we build the `lsblk` command to list the devices present on the node. Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
bfd5bfb
to
959ea33
Compare
jenkins test ceph-volume tox |
jenkins test ceph-volume all |
jenkins test ceph-volume lvm centos8-bluestore-create |
jenkins test ceph-volume lvm centos8-filestore-dmcrypt |
jenkins test ceph-volume lvm centos8-bluestore-dmcrypt |
jenkins test ceph-volume lvm centos8-filestore-create |
jenkins test ceph-volume lvm centos8-bluestore-dmcrypt |
Thanks @dsavineau. Sorry for the regression. 🤦♀️ |
This is a regression introduced by 9212420, when the host is using a
logical partition then lsblk reports that partition as a child from the
physical device.
That logical partition is prefixed by the
└─
character.This leads the
raw list
subcommand to show the lsblk error on the stderr.The lsblk command output looks like:
Using the
--list
option with lsblk solves the issue.Fixes: https://tracker.ceph.com/issues/52504
Signed-off-by: Dimitri Savineau dsavinea@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox