Landscape Client

Merge lp://staging/~tribaal/landscape-client/extra-logging-info into lp://staging/~landscape/landscape-client/trunk

  1. extra-logging-info
  2. Merge into trunk
Proposed by Chris Glass
Status: Merged
Approved by: Chris Glass
Approved revision: 777
Merged at revision: 775
Proposed branch: lp://staging/~tribaal/landscape-client/extra-logging-info
Merge into: lp://staging/~landscape/landscape-client/trunk
Diff against target: 135 lines (+27/-17)
2 files modified
landscape/broker/registration.py (+3/-3)
landscape/broker/tests/test_registration.py (+24/-14)
To merge this branch: bzr merge lp://staging/~tribaal/landscape-client/extra-logging-info
Reviewer Review Type Date Requested Status
Alberto Donato Approve
Free Ekanayaka (community) Approve
Review via email: mp+223711@code.staging.launchpad.net

Commit message

This branch switches to info instead of debug for the extra logging.

Description of the change

This branch switches to info instead of debug for the extra logging.

To post a comment you must log in.
lp://staging/~tribaal/landscape-client/extra-logging-info updated
776. By Chris Glass

Fixed tests checking log output

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

Thanks, +1!

review: Approve
Revision history for this message
Alberto Donato (ack) wrote :

+1, looks good!

#1:
Nitpick, feel free to ignore: Splitting log lines in tests so that "INFO" is at the beginning of the line would make them a bit more readable.

review: Approve
lp://staging/~tribaal/landscape-client/extra-logging-info updated
777. By Chris Glass

Split log assertions in a more readble way

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: