Francesco Romani has submitted this change and it was merged. Change subject: rpc: Log also error codes of RPC calls ......................................................................
rpc: Log also error codes of RPC calls In one of the previous commits, we added logging of RPC calls on info level. It'd be useful to have information about response codes there, so this patch adds that information. Change-Id: I776e667fcfd1a20a9ef5f6c638d6c3d950672314 Bug-Url: https://bugzilla.redhat.com/1351247 Backport-To: 4.0 Signed-off-by: Milan Zamazal <mzama...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/59234 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> Continuous-Integration: Jenkins CI Reviewed-on: https://gerrit.ovirt.org/59999 Reviewed-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/60103 --- M lib/yajsonrpc/__init__.py M vdsm/rpc/bindingxmlrpc.py 2 files changed, 36 insertions(+), 13 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Milan Zamazal: Verified -- To view, visit https://gerrit.ovirt.org/60103 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I776e667fcfd1a20a9ef5f6c638d6c3d950672314 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
