Dan Kenigsberg has submitted this change and it was merged. Change subject: Refactor v2v jobs for reusability ......................................................................
Refactor v2v jobs for reusability The jobs management infrastructure in v2v.py is general purpose and reusable. For example, storage would like to use the exact same thing to manage long-running storage operations. In order to make this possible, factor out the job management parts of v2v.py into a new module jobs.py and create a generic Job class which contains the complete interface needed by the job manager. In addition, add some basic tests for Job objects and their management. Change-Id: Ida6b1c460c5030c820c540e836e423d4632410df Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/44857 Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <from...@redhat.com> Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> --- M debian/vdsm.install M tests/Makefile.am A tests/jobsTests.py M vdsm.spec.in M vdsm/Makefile.am A vdsm/jobs.py 6 files changed, 313 insertions(+), 0 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Looks good to me, approved Adam Litke: Verified Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44857 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida6b1c460c5030c820c540e836e423d4632410df Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches