Merge lp://staging/~raharper/curtin/trunk.vmtest-logger-classname into lp://staging/~curtin-dev/curtin/trunk
Proposed by
Ryan Harper
| Status: | Merged |
|---|---|
| Merge reported by: | Scott Moser |
| Merged at revision: | not available |
| Proposed branch: | lp://staging/~raharper/curtin/trunk.vmtest-logger-classname |
| Merge into: | lp://staging/~curtin-dev/curtin/trunk |
| Diff against target: |
269 lines (+41/-32) 6 files modified
tests/vmtests/__init__.py (+11/-2) tests/vmtests/test_network.py (+20/-17) tests/vmtests/test_network_bonding.py (+1/-2) tests/vmtests/test_network_bridging.py (+2/-3) tests/vmtests/test_network_enisource.py (+4/-4) tests/vmtests/test_network_vlan.py (+3/-4) |
| To merge this branch: | bzr merge lp://staging/~raharper/curtin/trunk.vmtest-logger-classname |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Scott Moser (community) | Approve | ||
| Server Team CI bot | continuous-integration | Approve | |
|
Review via email:
|
|||
Description of the change
vmtest: initialize logger with class names for easy parsing
When running vmtest in parallel, having each message include the
TestClass name helps developers map the output to specific tests when
reading the combined log.
To post a comment you must log in.
Revision history for this message
| Server Team CI bot (server-team-bot) wrote | # |
review:
Approve
(continuous-integration)
Revision history for this message
| Scott Moser (smoser) | # |
review:
Approve
Revision history for this message
| Scott Moser (smoser) wrote | # |
Revision history for this message
| Scott Moser (smoser) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
PASSED: Continuous integration, rev:553
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild:
https:/