Change in vdsm[master]: lvm:moved metadatasize as an optional param for pv create in...

danken at redhat.com danken at redhat.com
Tue Mar 10 09:50:17 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: lvm:moved metadatasize as an optional param for pv create in lvm module
......................................................................


lvm:moved metadatasize as an optional param for pv create in lvm module

Allow metadatasize as an optional param for pvcreate. This feature is
required for gluster brick creation. Currently this param is not an
optional one.

Change-Id: I00f161c0d6996219553c556125a97a01d69a4be3
Signed-off-by: Timothy Asir <tjeyasin at redhat.com>
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38188
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/storage/lvm.py
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Timothy Asir: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/38188
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I00f161c0d6996219553c556125a97a01d69a4be3
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at gmail.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list