Merge lp:~le-chi-thu/lava-test/set-logging-level into lp:lava-test/0.0

Proposed by Le Chi Thu
Status: Merged
Merged at revision: 118
Proposed branch: lp:~le-chi-thu/lava-test/set-logging-level
Merge into: lp:lava-test/0.0
Diff against target: 11 lines (+1/-1)
1 file modified
lava_test/main.py (+1/-1)
To merge this branch: bzr merge lp:~le-chi-thu/lava-test/set-logging-level
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Paul Larson (community) Approve
Review via email: mp+91936@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

This makes sense to merge because it's a better default, but for a complete fix we should have a flag for this.

review: Approve
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

While this can go in now I think we'll likely want to defer to lava-tool for that task. I don't see any point of having more than one 'here is how to setup logging' code in LAVA.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lava_test/main.py'
2--- lava_test/main.py 2012-01-17 12:42:46 +0000
3+++ lava_test/main.py 2012-02-07 22:31:53 +0000
4@@ -44,7 +44,7 @@
5 FORMAT = '<LAVA_TEST>%(asctime)s %(levelname)s: %(message)s'
6 DATEFMT= '%Y-%m-%d %I:%M:%S %p'
7 logging.basicConfig(format=FORMAT,datefmt=DATEFMT)
8- logging.root.setLevel(logging.DEBUG)
9+ logging.root.setLevel(logging.ERROR)
10
11 run_with_dispatcher_class(LAVATestDispatcher)
12

Subscribers

People subscribed via source and target branches