Roy Golan has posted comments on this change.
Change subject: vdsm: report migration status in migrateStatuses()
......................................................................
Patch Set 7:
(1 comment)
....................................................
File vdsm_api/vdsmapi-schema.json
Line 2839: # Since: 4.10.0
Line 2840: ##
Line 2841: {'command': {'class': 'Host', 'name':
'getMigrationStatuses'},
Line 2842: 'data': {'*vmList': ['UUID']},
Line 2843: 'returns': ['MigrationStatus']}
this should return a dictionary of UUID: MigrationStatus otherwise how could the caller
corelate stats with a VM?
Line 2844:
Line 2845: ##
Line 2846: # @Host.ping:
Line 2847: #
--
To view, visit
http://gerrit.ovirt.org/15125
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc32fa00eaa1b90c3ba9d0afffdeb1cdd8150f9a
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <peet(a)redhat.com>
Gerrit-Reviewer: Better Saggi <bettersaggi(a)gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes