Francesco Romani has uploaded a new change for review.
Change subject: vmdsm: make use of the new onlyUUID parameter
......................................................................
vmdsm: make use of the new onlyUUID parameter
After RHBZ#1196735 VDSM gained a new parameter in
the 'getVMList' API, to ensure backward compatibility
with early 3.5.x Engines.
In VDSM, the {JSON,XML}RPC bindings make care to feed the API
with the right parameters.
MOM does access API directly, bypassing bindings, so must
take care of asking the right thing, ad that's done in this patch.
Change-Id: I7020c9bd182b0af7bfedb0d221025fd3c01abaca
Signed-off-by: Francesco Romani <[email protected]>
---
M mom/HypervisorInterfaces/vdsmInterface.py
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/mom refs/changes/09/38709/1
diff --git a/mom/HypervisorInterfaces/vdsmInterface.py
b/mom/HypervisorInterfaces/vdsmInterface.py
index 1193785..2603599 100644
--- a/mom/HypervisorInterfaces/vdsmInterface.py
+++ b/mom/HypervisorInterfaces/vdsmInterface.py
@@ -53,7 +53,7 @@
def getVmName(self, uuid):
try:
- response = self.vdsm_api.getVMList(True, [uuid])
+ response = self.vdsm_api.getVMList(True, [uuid], False)
self._check_status(response)
return response['vmList'][0]['vmName']
except vdsmException, e:
@@ -62,7 +62,7 @@
def getVmPid(self, uuid):
try:
- response = self.vdsm_api.getVMList(True, [uuid])
+ response = self.vdsm_api.getVMList(True, [uuid], False)
self._check_status(response)
return response['vmList'][0]['pid']
except vdsmException, e:
@@ -72,7 +72,7 @@
def getVmList(self):
vmIds = []
try:
- response = self.vdsm_api.getVMList()
+ response = self.vdsm_api.getVMList(onlyUUID=False)
self._check_status(response)
vm_list = response['vmList']
for vm in vm_list:
--
To view, visit https://gerrit.ovirt.org/38709
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7020c9bd182b0af7bfedb0d221025fd3c01abaca
Gerrit-PatchSet: 1
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
