Merge lp:~canonical-platform-qa/qa-jenkins-jobs/ust-rm-rf-venvs into lp:qa-jenkins-jobs

Proposed by Allan LeSage
Status: Merged
Approved by: Max Brustkern
Approved revision: 172
Merged at revision: 172
Proposed branch: lp:~canonical-platform-qa/qa-jenkins-jobs/ust-rm-rf-venvs
Merge into: lp:qa-jenkins-jobs
Diff against target: 22 lines (+3/-1)
2 files modified
jobs/ubuntu-system-tests/subunit-to-junit.sh (+1/-0)
jobs/ubuntu-system-tests/ubuntu-system-tests-builder.sh (+2/-1)
To merge this branch: bzr merge lp:~canonical-platform-qa/qa-jenkins-jobs/ust-rm-rf-venvs
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
platform-qa-bot continuous-integration Approve
Review via email: mp+303353@code.launchpad.net

Commit message

Build a new venv for every ust build.

Description of the change

This works around a weird new import error due to re-using of venvs and is more correct, IMO, to build a venv from scratch every time.

To post a comment you must log in.
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Max Brustkern (nuclearbob) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'jobs/ubuntu-system-tests/subunit-to-junit.sh'
2--- jobs/ubuntu-system-tests/subunit-to-junit.sh 2016-07-20 21:26:42 +0000
3+++ jobs/ubuntu-system-tests/subunit-to-junit.sh 2016-08-19 00:57:53 +0000
4@@ -1,3 +1,4 @@
5+rm -rf ve3
6 virtualenv --python=python3 ve3
7 . ve3/bin/activate
8
9
10=== modified file 'jobs/ubuntu-system-tests/ubuntu-system-tests-builder.sh'
11--- jobs/ubuntu-system-tests/ubuntu-system-tests-builder.sh 2016-08-09 20:31:45 +0000
12+++ jobs/ubuntu-system-tests/ubuntu-system-tests-builder.sh 2016-08-19 00:57:53 +0000
13@@ -22,7 +22,8 @@
14 cat $CONFIG_PATH
15
16 # let's work in a virtualenv
17-virtualenv --system-site-package --python=python3 ve3
18+rm -rf ve3
19+virtualenv --system-site-packages --python=python3 ve3
20 . ve3/bin/activate
21
22 export https_proxy=https://squid.internal:3128/

Subscribers

People subscribed via source and target branches