Merge lp:~roadmr/ols-tests/show-jobs-per-runner into lp:ols-tests

Proposed by Daniel Manrique
Status: Merged
Approved by: Daniel Manrique
Approved revision: 223
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~roadmr/ols-tests/show-jobs-per-runner
Merge into: lp:ols-tests
Diff against target: 12 lines (+3/-1)
1 file modified
olstests/filters.py (+3/-1)
To merge this branch: bzr merge lp:~roadmr/ols-tests/show-jobs-per-runner
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+331803@code.launchpad.net

Commit message

Show which tests got assigned to each runner.

This should help debug weird test-ordering-related problems that happen when using concurrency and for which the test order per runner is hard to reproduce locally.

Description of the change

Show which tests got assigned to each runner.

This should help debug weird test-ordering-related problems that happen when using concurrency and for which the test order per runner is hard to reproduce locally.

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Self-approving trivial (plus can be reverted later once I have debugged my problem)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'olstests/filters.py'
--- olstests/filters.py 2016-02-08 17:54:01 +0000
+++ olstests/filters.py 2017-10-04 17:46:52 +0000
@@ -112,7 +112,9 @@
112112
113 # Each suite is built taking each nth test, relying on filter_suite to113 # Each suite is built taking each nth test, relying on filter_suite to
114 # preserve the layout.114 # preserve the layout.
115 suites.append(filter_suite(nth, suite))115 this_runners_suite = filter_suite(nth, suite)
116 print("Assigned suite %s to runner %s" % (this_runners_suite, cur))
117 suites.append(this_runners_suite)
116 return suites118 return suites
117119
118120

Subscribers

People subscribed via source and target branches