Oved Ourfali has submitted this change and it was merged.
Change subject: getVMList: introducing onlyUUID parameter
......................................................................
getVMList: introducing onlyUUID parameter
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. We need to make sure that for newer engines vdsm sends map of
vmids and status and for older engines an array as it was before. In
order to achieve it we introduce new parameter 'onlyUUID' for getVMList
which is ignored by older vdsms and it makes sure that newer vdsms send
a map which fixes performance issue.
Change-Id: I51dea68e89a2c1685d16ba1921e1373eeb721860
Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com>
Bug-Url:
https://bugzilla.redhat.com/1198248
Related-To:
https://gerrit.ovirt.org/#/c/38449
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Piotr Kliczewski: Verified
Francesco Romani: Verified; Looks good to me, but someone else must approve
Omer Frenkel: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/38448
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I51dea68e89a2c1685d16ba1921e1373eeb721860
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5.2
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: automation(a)ovirt.org