Francesco Romani has posted comments on this change. Change subject: vm: switch to response.error() ......................................................................
Patch Set 6: (1 comment) https://gerrit.ovirt.org/#/c/38268/6//COMMIT_MSG Commit Message: Line 3: AuthorDate: 2015-02-23 18:22:11 +0100 Line 4: Commit: Francesco Romani <from...@redhat.com> Line 5: CommitDate: 2015-03-06 09:39:58 +0100 Line 6: Line 7: janitorial: vm: switch to response.error() > janitorial: -> vm: Done Line 8: Line 9: response.error() offers a nicer and cleaner Line 10: way to report error responses instead of the bare Line 11: errCode['something'], even when we are fine with -- To view, visit https://gerrit.ovirt.org/38268 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3825faf7a144ef8973dee3cb9f9f0e52fabfc039 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches