Dan Kenigsberg has uploaded a new change for review. Change subject: Revert "json-rpc: fix the Host.getVMList return value" ......................................................................
Revert "json-rpc: fix the Host.getVMList return value" This reverts commit e7ebfdeb7b036ce10f25a3b26657e6414644fc55. Unfortunately, this patch ended up causing a regression with oldready-released engine-3.5.0. Change-Id: I820a1f81e41bfe1c8f442db6d5d8d5b12de4ad79 Related-To: https://bugzilla.redhat.com/1198680 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm/rpc/Bridge.py 1 file changed, 8 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/38431/1 diff --git a/vdsm/rpc/Bridge.py b/vdsm/rpc/Bridge.py index e1781a2..e4b12aa 100644 --- a/vdsm/rpc/Bridge.py +++ b/vdsm/rpc/Bridge.py @@ -333,6 +333,13 @@ return API.Global().getVMList(True, vmList) +def Host_getVMList_Ret(ret): + """ + Just return a list of VM UUIDs + """ + return [v['vmId'] for v in ret['vmList']] + + def StoragePool_getInfo_Ret(ret): """ The result contains two data structures which must be merged @@ -402,7 +409,7 @@ 'Host_getStorageRepoStats': {'ret': Host_getStorageRepoStats_Ret}, 'Host_startMonitoringDomain': {}, 'Host_stopMonitoringDomain': {}, - 'Host_getVMList': {'call': Host_getVMList_Call, 'ret': 'vmList'}, + 'Host_getVMList': {'call': Host_getVMList_Call, 'ret': Host_getVMList_Ret}, 'Host_getVMFullList': {'call': Host_getVMFullList_Call, 'ret': 'vmList'}, 'Host_getAllVmStats': {'ret': 'statsList'}, 'Host_setupNetworks': {'ret': 'status'}, -- To view, visit https://gerrit.ovirt.org/38431 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I820a1f81e41bfe1c8f442db6d5d8d5b12de4ad79 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches