Francesco Romani has submitted this change and it was merged. Change subject: rpc: Log calls of API methods with possibly large results ......................................................................
rpc: Log calls of API methods with possibly large results We can prevent some API methods from logging their excessively large results. But this doesn't mean we shouldn't log their calls at all, we just shouldn't log the large values. This patch re-enables logging calls of the methods with suppressed logging, it just disables logging their outputs. Backport notice: this change is a cherry-pick from the ovirt-4.0 branch, but on that branch the change was reverted, and re-added later. This caused the changeid to be updated. Check both changeid and originalchangeid to properly track this change. Change-Id: I85d7f0ceee7e120081e2d9ad4ca3c8148ee70554 Original-Change-Id: I5a569f11146b5f3089207349d765a232c46e3b91 Bug-Url: https://bugzilla.redhat.com/1351247 Backport-To: 4.0 Backport-To: 3.6 Signed-off-by: Milan Zamazal <mzama...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/58464 Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <from...@redhat.com> Reviewed-by: Michal Skrivanek <michal.skriva...@redhat.com> Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/58709 Continuous-Integration: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/59993 Reviewed-on: https://gerrit.ovirt.org/60098 --- M lib/yajsonrpc/__init__.py M vdsm/rpc/bindingxmlrpc.py 2 files changed, 6 insertions(+), 6 deletions(-) Approvals: Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Milan Zamazal: Verified -- To view, visit https://gerrit.ovirt.org/60098 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85d7f0ceee7e120081e2d9ad4ca3c8148ee70554 Gerrit-PatchSet: 6 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: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
