Change in vdsm[master]: tests: Add FakeLVM PV properties tests

alitke at redhat.com alitke at redhat.com
Thu Nov 5 14:46:26 UTC 2015


Adam Litke has posted comments on this change.

Change subject: tests: Add FakeLVM PV properties tests
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/47858/4/tests/storagefakelib.py
File tests/storagefakelib.py:

Line 112: 
Line 113:     def getPV(self, pvName):
Line 114:         md = deepcopy(self.pvmd[pvName])
Line 115:         vg_name = md['vg_name']
Line 116:         md['vg_uuid'] = self.vgmd[vg_name]['uuid']
Update this to set the vg_uuid when vg is created.
Line 117:         return real_lvm.PV(**md)
Line 118: 
Line 119:     def getVG(self, vgName):
Line 120:         vg_md = deepcopy(self.vgmd[vgName])


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I629380923d4e3105a02b55df548d2a142c8def0a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list