Nir Soffer has submitted this change and it was merged. Change subject: rpc: Log RPC call summary on info level ......................................................................
rpc: Log RPC call summary on info level We currently log relatively detailed information about all RPC calls. This information is logged on the debug level. We are going to switch to the info logging level sometimes in future and then information about RPC calls won't be logged anymore. But we still want to see some very basic information about the RPC calls, such as that they were performed and how much time they took. This patch adds the corresponding logging. We additionally add the response to the context at a single place as a preparation for the following patch that logs the response. Change-Id: Idde2f1ba7394f16770543f5ca13411e8c2339cc6 Backport-To: 4.0 Signed-off-by: Milan Zamazal <[email protected]> Reviewed-on: https://gerrit.ovirt.org/59080 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Continuous-Integration: Jenkins CI --- M lib/vdsm/rpc/bindingxmlrpc.py M lib/yajsonrpc/__init__.py 2 files changed, 26 insertions(+), 11 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Looks good to me, approved Jenkins CI: Passed CI tests Milan Zamazal: Verified -- To view, visit https://gerrit.ovirt.org/59080 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idde2f1ba7394f16770543f5ca13411e8c2339cc6 Gerrit-PatchSet: 17 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> 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]
