Change in vdsm[master]: [wip] jsonrpc: Vdsm changes
alitke at redhat.com
alitke at redhat.com
Thu Dec 5 14:45:44 UTC 2013
Adam Litke has posted comments on this change.
Change subject: [wip] jsonrpc: Vdsm changes
......................................................................
Patch Set 8: Code-Review-1
(2 comments)
....................................................
File vdsm_api/vdsmapi-schema.json
Line 2932
Line 2933
Line 2934
Line 2935
Line 2936
How would you describe that this could return either ['UUID'] or ['VmDescription'] ?
Line 2951: #
Line 2952: # @vmList: #optional Filter the results by a list of UUIDs
Line 2953: #
Line 2954: # @fullStatus: #optional Flag which detremines whether to return full status
Line 2955: #
This is a bad idea. One major cleanup that this schema has imposed is that a function has a stable return type. It is not good to have a parameter that can change the return value from a list of UUIDs to a list of VmDescriptions.
Line 2956: # Returns:
Line 2957: # A list of VM UUIDs
Line 2958: #
Line 2959: # Since: 4.10.0
--
To view, visit http://gerrit.ovirt.org/19497
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If828355b7efe28fe6a2e784069425fefd2f3f25c
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list