Nir Soffer has submitted this change and it was merged. Change subject: jobs: Allow run and abort only from valid states ......................................................................
jobs: Allow run and abort only from valid states Before this patch we allow abort and run from any state even when it does not make sense. Abort is valid only for pending or running jobs and run is valid only for pending jobs. When an invalid call is attempted raise a new error. Change-Id: Ia8bc1ab264c5786b43fb409c3fab8913337778bf Signed-off-by: Adam Litke <[email protected]> Reviewed-on: https://gerrit.ovirt.org/63711 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Jenkins CI --- M lib/vdsm/define.py M lib/vdsm/exception.py M lib/vdsm/jobs.py M tests/jobsTests.py 4 files changed, 47 insertions(+), 4 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/63711 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8bc1ab264c5786b43fb409c3fab8913337778bf Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
