Merge lp://staging/~rodsmith/checkbox/multi-iperf-targets into lp://staging/checkbox
Proposed by
Rod Smith
| Status: | Merged |
|---|---|
| Approved by: | Jeff Lane |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 4115 |
| Proposed branch: | lp://staging/~rodsmith/checkbox/multi-iperf-targets |
| Merge into: | lp://staging/checkbox |
| Diff against target: |
183 lines (+60/-40) 1 file modified
providers/plainbox-provider-checkbox/bin/network (+60/-40) |
| To merge this branch: | bzr merge lp://staging/~rodsmith/checkbox/multi-iperf-targets |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jeff Lane | Approve | ||
| Daniel Manrique (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Enable network test to support multiple iperf/iperf3 targets. Aside from some changes to logging output, this should work exactly the same way as before when fed a single target. When fed multiple targets via a comma-delimited list in TEST_TARGET_IPERF, the test now tries each one in turn until one succeeds or until all the potential targets fail. This change is motivated to support server environments with separate 1Gbps and 10Gbps networks.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks OK to me, thanks.