From Dan Kenigsberg <dan...@redhat.com>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: pylint: drop awkward logging.TRACE level ...................................................................... pylint: drop awkward logging.TRACE level Writing into the logging module is rude, and not really helpful. Our default log level is INFO, so the few places that were using TRACE can safely use DEBUG. Change-Id: Iae630e3a26c000f3553b4771a1c1f53248fd6af0 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> --- M README.logging M lib/vdsm/virt/guestagent.py M lib/vdsm/virt/vmchannels.py M tests/guestagentTests.py M vdsm/vdsmd.py 5 files changed, 11 insertions(+), 21 deletions(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Looks good to me, but someone else must approve Dan Kenigsberg: Verified Francesco Romani: Looks good to me, approved Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/69780 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae630e3a26c000f3553b4771a1c1f53248fd6af0 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org