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

luminous: ceph-volume: fix zap not working with LVs #24082

Merged
merged 3 commits into from Sep 13, 2018

Conversation

Projects
None yet
3 participants
@alfredodeza
Contributor

alfredodeza commented Sep 13, 2018

alfredodeza added some commits Sep 13, 2018

ceph-volume lvm.zap use remove_pv only for physical devices
Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit 036683f)
ceph-volume tests.functional use --destroy when zapping lvs in bluestore
Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit 87af815)
ceph-volume tests.functional comment why not use --destroy when zappi…
…ng lvs in filestore

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit 11858c7)

@ceph-jenkins ceph-jenkins added this to the luminous milestone Sep 13, 2018

@alfredodeza alfredodeza requested a review from andrewschoen Sep 13, 2018

@andrewschoen andrewschoen merged commit 913a1cc into ceph:luminous Sep 13, 2018

25 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
ceph-volume batch testing centos7-bluestore-mixed_type OK
Details
ceph-volume batch testing centos7-bluestore-mixed_type_dmcrypt OK
Details
ceph-volume batch testing centos7-bluestore-single_type OK
Details
ceph-volume batch testing centos7-bluestore-single_type_dmcrypt OK
Details
ceph-volume batch testing centos7-filestore-mixed_type OK
Details
ceph-volume batch testing centos7-filestore-mixed_type_dmcrypt OK
Details
ceph-volume batch testing centos7-filestore-single_type OK
Details
ceph-volume batch testing centos7-filestore-single_type_dmcrypt OK
Details
ceph-volume batch testing xenial-bluestore-single_type OK
Details
ceph-volume batch testing xenial-bluestore-single_type_dmcrypt OK
Details
ceph-volume batch testing xenial-filestore-single_type OK
Details
ceph-volume batch testing xenial-filestore-single_type_dmcrypt OK
Details
ceph-volume lvm testing centos7-bluestore-create OK
Details
ceph-volume lvm testing centos7-bluestore-dmcrypt OK
Details
ceph-volume lvm testing centos7-filestore-create OK
Details
ceph-volume lvm testing centos7-filestore-dmcrypt OK
Details
ceph-volume lvm testing xenial-bluestore-create OK
Details
ceph-volume lvm testing xenial-bluestore-dmcrypt OK
Details
ceph-volume lvm testing xenial-filestore-create OK
Details
ceph-volume lvm testing xenial-filestore-dmcrypt OK
Details
ceph-volume tox tests OK
Details
make check make check succeeded
Details

@alfredodeza alfredodeza deleted the alfredodeza:luminous-rm35970 branch Sep 19, 2018

@smithfarm smithfarm changed the title from luminous: ceph-volume fix zap not working with LVs to luminous: ceph-volume: fix zap not working with LVs Oct 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment