[node-patches] Change in ovirt-node[master]: log: Add a log module

fabiand at fedoraproject.org fabiand at fedoraproject.org
Tue Sep 24 11:01:14 UTC 2013


Fabian Deutsch has submitted this change and it was merged.

Change subject: log: Add a log module
......................................................................


log: Add a log module

Previously the log configuration wasn't consistent. This was due to the
fact that the order of imports determined what class got what logging
configuration. Now a central module handles the configuration and logger
initialization to prevent some bugs.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=991267
Change-Id: Ica7d81cc9ab73acc4feaab474db3c8ac4258fc42
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/app.py
M src/ovirt/node/base.py
M src/ovirt/node/config/defaults.py
M src/ovirt/node/config/network.py
M src/ovirt/node/installer/__main__.py
A src/ovirt/node/log.py
M src/ovirt/node/plugins.py
M src/ovirt/node/setup/__main__.py
M src/ovirt/node/ui/widgets.py
M src/ovirt/node/utils/fs.py
M src/ovirt/node/utils/input.py
M src/ovirt/node/utils/network.py
M src/ovirt/node/utils/process.py
M src/ovirt/node/utils/system.py
14 files changed, 136 insertions(+), 97 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/18927
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ica7d81cc9ab73acc4feaab474db3c8ac4258fc42
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list