Change in vdsm[master]: API: onlyUUID should affect only short status

fromani at redhat.com fromani at redhat.com
Fri Mar 13 08:27:41 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: API: onlyUUID should affect only short status
......................................................................


Patch Set 2: Code-Review-1

preliminary verification failed.

MOM also calls getVMList() without arguments. The current defaults are:

fullStatus=False, vmList=(), onlyUUID=True

so MOM will get the 3.5.0 short status, which is no good. We need to
- switch the default for onlyUUID
- make sure that at Binding level (XMLRPC, JSONRPC) code passes always the right combination of parameters to keep Engine happy (and not break _again_ something else).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id05a1bb92100023e8a96ee6f13daa8eedd233ff6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list