Francesco Romani has submitted this change and it was merged. Change subject: API: logging: move setLogLevel code into logUtils ......................................................................
API: logging: move setLogLevel code into logUtils Move the implementation of API.setLogLevel into the logUtils module, with minimal changes and renaming, to make it less coupled. The rationale for this move is that we should not have logic in API.py besides the bare minimum. Furthermore, setLogLevel fits nicely in the logUtils domain. Change-Id: Iacda9f82c6e0bc22a19403b99acbfffbdb7fd2bf Backport-To: 4.0 Backport-To: 3.6 Bug-Url: https://bugzilla.redhat.com/1351077 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/58990 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/59872 Reviewed-by: Martin Polednik <[email protected]> Reviewed-by: Milan Zamazal <[email protected]> --- M lib/vdsm/logUtils.py M vdsm/API.py 2 files changed, 10 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Passed CI tests Francesco Romani: Verified; Looks good to me, approved Milan Zamazal: Looks good to me, but someone else must approve Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59872 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacda9f82c6e0bc22a19403b99acbfffbdb7fd2bf Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
