Nir Soffer has posted comments on this change.

Change subject: StorageDomain.getInfo - report lvm metadata device for block sd
......................................................................


Patch Set 5:

(2 comments)

https://gerrit.ovirt.org/#/c/64433/5/lib/api/vdsm-api.yml
File lib/api/vdsm-api.yml:

Line 5659:             type: string
Line 5660: 
Line 5661:         -   defaultvalue: null
Line 5662:             description: The GUID of the device containing the 
domain lvm metadata (optional)
Line 5663:             name: lvmmetadatadevice
> Update names
vgMetadataDevice
Line 5664:             type: string
Line 5665:         type: object
Line 5666: 
Line 5667:     StorageDomainStatus: &StorageDomainStatus


https://gerrit.ovirt.org/#/c/64433/5/vdsm/storage/blockSD.py
File vdsm/storage/blockSD.py:

Line 1116: <<<<<<< HEAD
Line 1117:         info['metadatadevice'] = self._manifest.getMetadataLVDevice()
Line 1118: =======
Line 1119:         info['metadatadevice'] = 
self._manifest.getMetadataLVFirstDevice()
Line 1120:         info['lvmmetadatadevice'] = 
self._manifest.getVgMetadataDevice()
> And this, but
Use mixed  case name, already used in this verb response.
Line 1121: >>>>>>> 7da51c9... StorageDomain.getInfo - report lvm metadata 
device for block sd
Line 1122:         return info
Line 1123: 
Line 1124:     def getStats(self):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4a7763d2ab7d796be633ecd69f661cba96e29dde
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to