Merge lp:~canonical-platform-qa/snappy-ecosystem-tests/no-ephemeral-container into lp:snappy-ecosystem-tests

Proposed by Omer Akram
Status: Merged
Approved by: Omer Akram
Approved revision: 53
Merged at revision: 53
Proposed branch: lp:~canonical-platform-qa/snappy-ecosystem-tests/no-ephemeral-container
Merge into: lp:snappy-ecosystem-tests
Diff against target: 10 lines (+1/-1)
1 file modified
snappy_ecosystem_tests/environment/data/snapd.py (+1/-1)
To merge this branch: bzr merge lp:~canonical-platform-qa/snappy-ecosystem-tests/no-ephemeral-container
Reviewer Review Type Date Requested Status
platform-qa-bot continuous-integration Approve
Heber Parrucci (community) Approve
Review via email: mp+321884@code.launchpad.net

Commit message

Don't use ephemeral container

Description of the change

Don't use ephemeral container, we are not using any of its advantage. Its causing a problem when stopping container, says "container X already exists".

Test results here: https://platform-qa-jenkins.ubuntu.com/job/run_snappy_ecosystem_tests/161/testReport/

Note the 100% success rate is combined with https://code.launchpad.net/~canonical-platform-qa/snappy-ecosystem-tests/dont-install-chromedriver

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
Heber Parrucci (heber013) wrote :

LGTM

review: Approve
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'snappy_ecosystem_tests/environment/data/snapd.py'
2--- snappy_ecosystem_tests/environment/data/snapd.py 2017-03-15 12:37:04 +0000
3+++ snappy_ecosystem_tests/environment/data/snapd.py 2017-04-04 16:03:53 +0000
4@@ -29,5 +29,5 @@
5 "protocol": "simplestreams",
6 "fingerprint": "16.04"
7 },
8- "ephemeral": True
9+ "ephemeral": False
10 }

Subscribers

People subscribed via source and target branches