QA Regression Testing

Merge ~jslarraz/qa-regression-testing:add_test_iperf3 into qa-regression-testing:master

  1. Git
  2. lp:~jslarraz/qa-regression-testing
  3. add_test_iperf3
  4. Merge into master
Proposed by Jorge Sancho Larraz
Status: Merged
Merged at revision: 42d9f05032637862fc4d825fb6c07006665ef9ca
Proposed branch: ~jslarraz/qa-regression-testing:add_test_iperf3
Merge into: qa-regression-testing:master
Diff against target: 133 lines (+111/-0)
2 files modified
.launchpad.yaml (+14/-0)
scripts/test-iperf3.py (+97/-0)
Reviewer Review Type Date Requested Status
Steve Beattie (community) Approve
Alex Murray (community) Approve
Review via email: mp+453895@code.staging.launchpad.net

Commit message

Add test-iperf3.py

Description of the change

Create new tests for iperf3 package

To post a comment you must log in.
Revision history for this message
Alex Murray (alexmurray) wrote :

LGTM but it would be good to add this to .launchpad.yaml so that it gets run by lpci automatically as well against each of the stable releases.

review: Approve
Revision history for this message
Steve Beattie (sbeattie) wrote :

This is great, thanks fo rgetting the infrastrucutre in place to make this test workable and preparing these test. It's fine to merge as is, but a couple of minor comments inline.

And adding it to the .launchpad.yaml would be appreciated, peek at the history for the file for examples, and https://git.launchpad.net/qa-regression-testing/tree/scripts/README.md#n144 also covers a bit of documentation as well as how to handle cases that can't be run in lpci. (That said, please help clarify the documentation.)

THanks!

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

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