Dan Kenigsberg has submitted this change and it was merged.
Change subject: resize-lun: Support guids parameter in getDeviceList
......................................................................
resize-lun: Support guids parameter in getDeviceList
Add optional "guids" parameter to getDeviceList. If specified,
getDevicesList will get info and test only these devices.
In the resize LUN flow, we need to rescan and resize the devices before
resizing the PV. Engine will invoke getDeviceList verb with specific
LUNs ids.
In getDeviceList we test PV creation and this is a heavyweight
operation, using the guids parameter we limit the operation to the
specified devices.
Examples:
getDeviceList
return all devices
getDeviceList FCP
return only FCP devices
getDeviceList ISCSI guid1 guid2
return info for guid1 and guid2, assuming ISCSI type
Change-Id: Ic173d94a132e617ae97353d38520a86bede657d7
Bug-Url:https://bugzilla.redhat.com/609689
Signed-off-by: Fred Rolland <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/40661
Reviewed-by: Nir Soffer <[email protected]>
Tested-by: Nir Soffer <[email protected]>
Reviewed-by: Jenkins CI
Continuous-Integration: Jenkins CI
Tested-by: Jenkins CI
Reviewed-by: Adam Litke <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
---
M client/vdsClient.py
M vdsm/API.py
M vdsm/rpc/bindingxmlrpc.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/hsm.py
5 files changed, 29 insertions(+), 11 deletions(-)
Approvals:
Adam Litke: Looks good to me, approved
Nir Soffer: Verified; Looks good to me, but someone else must approve
Federico Simoncelli: Looks good to me, approved
Jenkins CI: Verified; Looks good to me, but someone else must approve; Passed
CI tests
--
To view, visit https://gerrit.ovirt.org/40661
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic173d94a132e617ae97353d38520a86bede657d7
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Ala Hino <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Fred Rolland <[email protected]>
Gerrit-Reviewer: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
