Dan Kenigsberg has submitted this change and it was merged. Change subject: lvm: Allow multiple LVs to be refreshed at once. ......................................................................
lvm: Allow multiple LVs to be refreshed at once. This change refactors the refreshLV method to allow multiple volumes to be refreshed at once as possible with the activate and deactive methods. This is patch 1 in a series of 3 that aim to resolve BZ#1009812. This patch is required by the changes listed below : lvm: Make the isLVActive() method public. http://gerrit.ovirt.org/#/c/21386/ blockSD: Ensure active volumes are refreshed before use. http://gerrit.ovirt.org/#/c/21387 bug-url: https://bugzilla.redhat.com/1009812 Change-Id: I1ecc64b8ca0133b030ba5bfa37f1a2c55067dd5d Signed-off-by: Lee Yarwood <lyarw...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/19871 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Allon Mureinik <amure...@redhat.com> Tested-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Federico Simoncelli <fsimo...@redhat.com> --- M vdsm/storage/blockVolume.py M vdsm/storage/lvm.py 2 files changed, 6 insertions(+), 5 deletions(-) Approvals: Nir Soffer: Verified; Looks good to me, but someone else must approve Federico Simoncelli: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/19871 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ecc64b8ca0133b030ba5bfa37f1a2c55067dd5d Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lee Yarwood <lyarw...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Eduardo <ewars...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Lee Yarwood <lyarw...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches