Change in vdsm[master]: API: Add api for getting host jobs info

nsoffer at redhat.com nsoffer at redhat.com
Tue Dec 15 10:48:52 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: API: Add api for getting host jobs info
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/49451/4/vdsm/API.py
File vdsm/API.py:

Line 1463:         return v2v.convert_ova(ova_path, vminfo, jobid, self._cif.irs)
Line 1464: 
Line 1465:     def getJobs(self, job_type=None, job_ids=()):
Line 1466:         return dict(status=doneCode,
Line 1467:                     jobs=jobs.info(job_type=job_type, job_ids=job_ids))
> we have the response module for this kind of task. Is it gone out of favour
Fixed in current version
Line 1468: 
Line 1469:     def getConvertedVm(self, jobid):
Line 1470:         return v2v.get_converted_vm(jobid)
Line 1471: 


-- 
To view, visit https://gerrit.ovirt.org/49451
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I957938ea9ca55d2499d09dfed39821c85e3e9d48
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list