Merge lp:~canonical-platform-qa/qa-jenkins-jobs/just-enough-dependencies into lp:qa-jenkins-jobs

Proposed by Allan LeSage
Status: Merged
Approved by: Max Brustkern
Approved revision: 54
Merged at revision: 55
Proposed branch: lp:~canonical-platform-qa/qa-jenkins-jobs/just-enough-dependencies
Merge into: lp:qa-jenkins-jobs
Diff against target: 11 lines (+1/-1)
1 file modified
scripts/ubuntu-system-tests/run-ubuntu-system-tests.sh (+1/-1)
To merge this branch: bzr merge lp:~canonical-platform-qa/qa-jenkins-jobs/just-enough-dependencies
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Review via email: mp+292439@code.launchpad.net

Commit message

Get "just enough" dependencies.

Description of the change

Quickie change to take advantage of the new ust -d and -p features.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

So we need the production dependencies but not the test dependencies? The nomenclature is confusing, but I'm wrapping my head around it.

Revision history for this message
Allan LeSage (allanlesage) wrote :

I guess I agree that the wording could be clearer but we're just the clients of the ubuntu-system-tests from this perspective, should I add a comment to clarify? Else we can file a bug against that project.

Revision history for this message
Max Brustkern (nuclearbob) wrote :

Got it!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scripts/ubuntu-system-tests/run-ubuntu-system-tests.sh'
2--- scripts/ubuntu-system-tests/run-ubuntu-system-tests.sh 2016-04-18 01:48:41 +0000
3+++ scripts/ubuntu-system-tests/run-ubuntu-system-tests.sh 2016-04-20 20:25:43 +0000
4@@ -20,7 +20,7 @@
5 bzr branch $UBUNTU_SYSTEM_TESTS_BRANCH ubuntu-system-tests
6 cd ubuntu-system-tests
7 python3 setup.py develop
8-python3 -m ubuntu_system_tests.command_line -n -i -u -v --config=$CONFIG_PATH "$TESTS"
9+python3 -m ubuntu_system_tests.command_line -n -d -p -u -v --config=$CONFIG_PATH "$TESTS"
10
11 deactivate # our virtualenv
12

Subscribers

People subscribed via source and target branches