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

agl at us.ibm.com agl at us.ibm.com
Tue Apr 17 13:25:44 UTC 2012


Adam Litke has uploaded a new change for review.

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

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

As discussed on the list, I found that the getAllTasks* APIs do not return all
tasks but instead filter to return only spm tasks.  This is a requirement for
ovirt-engine.  To make it possible to view all tasks while preserving
compatibility for ovirt-engine I propose the following changes to the API:

The public API in API.py will be modified to add a 'tag' parameter which
defaults to None.  This will be passed all the way down into the taskManager
(which already expects such a parameter).  In this way, the new default mode of
operation will be fixed to return all tasks as the name of the API states.

To maintain compatibility for ovirt-engine, the XMLRPC binding will pass the tag
"spm" to these APIs to get the current filtering behavior.  Note that I have
also changed vdsClient so that it will return all tasks.

Change-Id: I43371d358dfdfcfcee14aca40ed06d63be7c0a44
Signed-off-by: Adam Litke <agl at us.ibm.com>
---
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/storage/hsm.py
M vdsm_cli/vdsClient.py
4 files changed, 33 insertions(+), 19 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/3613/1
--
To view, visit http://gerrit.ovirt.org/3613
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I43371d358dfdfcfcee14aca40ed06d63be7c0a44
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <agl at us.ibm.com>


More information about the vdsm-patches mailing list