Change in vdsm[ovirt-3.3]: lvm: Allow multiple LVs to be refreshed at once

fsimonce at redhat.com fsimonce at redhat.com
Thu Nov 21 16:18:52 UTC 2013


Federico Simoncelli 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

lvm.refreshLV() function used to accept only single lv, requiring
multiple invocations for refreshing multiple lvs, although lvm allows
refreshing of multiple lvs. This patch rename the function and accept
multiple lvs and invoke lvchange once with given lvs.

This change is required for patch http://gerrit.ovirt.org/#/c/21501.

Change-Id: I1ecc64b8ca0133b030ba5bfa37f1a2c55067dd5d
Bug-Url: https://bugzilla.redhat.com/1009812
Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/19871
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Tested-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21500
---
M vdsm/storage/blockVolume.py
M vdsm/storage/lvm.py
2 files changed, 6 insertions(+), 5 deletions(-)

Approvals:
  Nir Soffer: Verified
  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/21500
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ecc64b8ca0133b030ba5bfa37f1a2c55067dd5d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list