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