Change in vdsm[master]: API: getAllTasks{Info|Statuses} does not return all tasks

danken at redhat.com danken at redhat.com
Mon Apr 23 20:50:07 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: API: getAllTasks{Info|Statuses} does not return all tasks
......................................................................


Patch Set 1: (2 inline comments)

Doron, could you give us a hand? I would like to verify that we do not break an API that an old-but-installed rhevm^WEngine depends upon.

....................................................
File vdsm_cli/vdsClient.py
Line 1107:             tag = ''
for the record, I don't understand Ayal either.

....................................................
File vdsm/storage/hsm.py
Line 1691:     def getAllTasksStatuses(self, tag=None, options = None):
I'd try recruiting Doron's memory

--
To view, visit http://gerrit.ovirt.org/3613
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I43371d358dfdfcfcee14aca40ed06d63be7c0a44
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list