Nir Soffer has submitted this change and it was merged. Change subject: jobs: Fix job error format ......................................................................
jobs: Fix job error format The schema claims that job's error (ErrorInfo) is a map with "code" and "message" keys, but we returned an error response dict used for xmlrpc response. Add GeneralException.info() method, and use it to return the promised value. Change-Id: I2fdc6db6ed7519e5860d34e3a602dbeecce9a998 Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/51145 Continuous-Integration: Jenkins CI Reviewed-by: Liron Aravot <lara...@redhat.com> --- M lib/vdsm/jobs.py M lib/vdsm/utils.py M tests/jobsTests.py 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Nir Soffer: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Liron Aravot: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/51145 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2fdc6db6ed7519e5860d34e3a602dbeecce9a998 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches