Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#846323 - Search PV's belonging to the VG in removeVG. ......................................................................
BZ#846323 - Search PV's belonging to the VG in removeVG. Use only one lvm operation when removing a VG. Using more than one is racy. In addition avoid the use of getAllPvs() which will be removed in the future. Only one getAllPvs() call remains! Change-Id: Ica565f74774fd1dcce7c18361aef5e1464c78b68 Signed-off-by: Eduardo <[email protected]> --- M vdsm/storage/lvm.py 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Ayal Baron: Looks good to me, approved Eduardo: Verified Dan Kenigsberg: -- To view, visit http://gerrit.ovirt.org/7066 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica565f74774fd1dcce7c18361aef5e1464c78b68 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Eduardo <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Paikov <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Haim Ateya <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
