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 <[email protected]> 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 <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
