Merge lp:~brian.curtin/ubuntuone-control-panel/updater-logging into lp:ubuntuone-control-panel
Proposed by
Brian Curtin
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Brian Curtin | ||||
Approved revision: | 402 | ||||
Merged at revision: | 394 | ||||
Proposed branch: | lp:~brian.curtin/ubuntuone-control-panel/updater-logging | ||||
Merge into: | lp:ubuntuone-control-panel | ||||
Diff against target: |
171 lines (+37/-27) 2 files modified
bin/ubuntuone-updater (+35/-25) ubuntuone/controlpanel/tests/test_web_client.py (+2/-2) |
||||
To merge this branch: | bzr merge lp:~brian.curtin/ubuntuone-control-panel/updater-logging | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
dobey (community) | Approve | ||
Paul Hummer (community) | Approve | ||
Mike McCracken (community) | Approve | ||
Review via email: mp+145030@code.launchpad.net |
Commit message
- Use logging functionality instead of printing messages to stdout (lp:1105447)
Description of the change
In order to make it easy (or even possible) to figure out what the updater is doing in a real environment, we need to move away from printing and to logging.
To post a comment you must log in.
Looks good, works for me on osx CLI