ols-tests

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

  1. show-jobs-per-runner
  2. Merge into trunk
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
1=== modified file 'olstests/filters.py'
2--- olstests/filters.py 2016-02-08 17:54:01 +0000
3+++ olstests/filters.py 2017-10-04 17:46:52 +0000
4@@ -112,7 +112,9 @@
5
6 # Each suite is built taking each nth test, relying on filter_suite to
7 # preserve the layout.
8- suites.append(filter_suite(nth, suite))
9+ this_runners_suite = filter_suite(nth, suite)
10+ print("Assigned suite %s to runner %s" % (this_runners_suite, cur))
11+ suites.append(this_runners_suite)
12 return suites
13
14

Subscribers

People subscribed via source and target branches