snappy-hwe-snaps

Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/build-scripts:remove-containers into ~snappy-hwe-team/snappy-hwe-snaps/+git/build-scripts:master

  1. Git
  2. lp:~alfonsosanchezbeato/snappy-hwe-snaps/+git/build-scripts
  3. remove-containers
  4. Merge into master
Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Konrad Zapałowicz
Approved revision: 4312492384e80ea69c5224706669061ff3a89eb1
Merged at revision: 0b9ed0685a50a5a708c50ebe2ab0f7eee25f98f6
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/build-scripts:remove-containers
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/build-scripts:master
Diff against target: 30 lines (+8/-4)
2 files modified
jobs/generic-cleanup-snap (+3/-0)
scripts/test-snap.sh (+5/-4)
Reviewer Review Type Date Requested Status
Konrad Zapałowicz (community) Approve
System Enablement Bot continuous-integration Approve
Simon Fels Approve
Review via email: mp+322775@code.launchpad.net

Commit message

Cleanup containers after usage

Description of the change

Cleanup containers after usage

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

PASSED: Continuous integration, rev:2505905dbd4bdece427369364a8f404ed7f6dcc3
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1472/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-build-snap-worker/1151
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/1380/console
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-test-snap/1639

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1472/rebuild

review: Approve (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

Branch refreshed.

Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

PASSED: Continuous integration, rev:cd634dcea94ce9ba7aa33e65c15452f601090586
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1473/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-build-snap-worker/1152
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/1381/console
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-test-snap/1640

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1473/rebuild

review: Approve (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

PASSED: Continuous integration, rev:4312492384e80ea69c5224706669061ff3a89eb1
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1474/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-build-snap-worker/1153
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/1382/console
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/generic-test-snap/1641

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/1474/rebuild

review: Approve (continuous-integration)
Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

ack

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/jobs/generic-cleanup-snap b/jobs/generic-cleanup-snap
index a260fe1..54fcc00 100755
--- a/jobs/generic-cleanup-snap
+++ b/jobs/generic-cleanup-snap
@@ -27,3 +27,6 @@ $WORKSPACE/build-scripts/scripts/delete-ci-repo.py \
27 --git-repo=https://git.launchpad.net/~snappy-hwe-team/snappy-hwe-snaps/+git/$CI_REPO27 --git-repo=https://git.launchpad.net/~snappy-hwe-team/snappy-hwe-snaps/+git/$CI_REPO
2828
29$SSH rm -rf $REMOTE_RESULTS_BASE_DIR/$RESULTS_ID29$SSH rm -rf $REMOTE_RESULTS_BASE_DIR/$RESULTS_ID
30
31# Now remove any container that might have been left behind...
32$SSH sudo docker rm $(sudo docker ps -q --filter=status=exited --filter=ancestor=snap-spread-tests)
diff --git a/scripts/test-snap.sh b/scripts/test-snap.sh
index b756c9b..6c0f04f 100755
--- a/scripts/test-snap.sh
+++ b/scripts/test-snap.sh
@@ -85,10 +85,11 @@ EOF
85 $SSH time sudo docker build -t snap-spread-tests "$REMOTE_WORKSPACE"/docker85 $SSH time sudo docker build -t snap-spread-tests "$REMOTE_WORKSPACE"/docker
8686
87 $SSH time sudo docker run \87 $SSH time sudo docker run \
88 -v /dev:/dev \88 --rm \
89 -v "$REMOTE_WORKSPACE":/build \89 -v /dev:/dev \
90 --privileged \90 -v "$REMOTE_WORKSPACE":/build \
91 snap-spread-tests /build/run-tests.sh91 --privileged \
92 snap-spread-tests /build/run-tests.sh
9293
93 $SSH sudo rm -rf "$REMOTE_WORKSPACE"94 $SSH sudo rm -rf "$REMOTE_WORKSPACE"
9495

Subscribers

People subscribed via source and target branches