Nir Soffer has submitted this change and it was merged. Change subject: jobs: Add error property ......................................................................
jobs: Add error property When a Job encounters an error, there should be a standardized way for that error to be stored so that code working with Job objects has a standard way to propagate the error. Add an 'error' property to the job class which is None by default but can contain an instance of utils.GeneralException or one of its decendants. If error is set, the dictionary returned by the call to info() will contain an 'error' key. The value of this key is a status code (containing a standard vdsm code and message). Change-Id: I3f6e27beeeecd69db1673e5168524b0ce94f2e9e Signed-off-by: Adam Litke <[email protected]> Reviewed-on: https://gerrit.ovirt.org/50354 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Shahar Havivi <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Continuous-Integration: Jenkins CI --- M lib/vdsm/jobs.py M tests/jobsTests.py 2 files changed, 25 insertions(+), 7 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Adam Litke: Verified Nir Soffer: Looks good to me, approved Shahar Havivi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50354 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f6e27beeeecd69db1673e5168524b0ce94f2e9e Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Pablo Iranzo Gómez Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Shahar Havivi <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
