Change in vdsm[ovirt-3.5]: api: introducing onlyUUID parameter for getVMList verb

ybronhei at redhat.com ybronhei at redhat.com
Mon Mar 9 12:21:02 UTC 2015


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: api: introducing onlyUUID parameter for getVMList verb
......................................................................


api: introducing onlyUUID parameter for getVMList verb

Due to vdsm sending only an array of vmids in a response to getVMList
the engine asks separately using VM.getStats which creates unnecessary
load. In order to fix this issue we introduce a new parameter 'onlyUUID'
which is set to True by default. Engine 3.5.0 still wants to get an
array of vmids, but for all other versions we set 'onlyUUID' to False
which returns map of vmids and statuses.

Change-Id: I2d9eb359f6995c09d61a3159e733498fa3b55780
Bug-Url: https://bugzilla.redhat.com/1196735
Related-To: https://gerrit.ovirt.org/#/c/38448
Signed-off-by: pkliczewski <piotr.kliczewski at gmail.com>
Reviewed-on: https://gerrit.ovirt.org/38476
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Tested-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/API.py
M vdsm/rpc/BindingXMLRPC.py
M vdsm/rpc/Bridge.py
M vdsm/rpc/vdsmapi-schema.json
4 files changed, 20 insertions(+), 11 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Yaniv Bronhaim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2d9eb359f6995c09d61a3159e733498fa3b55780
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list