ubuntu-openstack-ci

Merge lp:~fnordahl/ubuntu-openstack-ci/ubuntu-openstack-ci into lp:ubuntu-openstack-ci

  1. ubuntu-openstack-ci
  2. Merge into trunk
Proposed by Frode Nordahl
Status: Merged
Approved by: Chris MacNaughton
Approved revision: 427
Merged at revision: 427
Proposed branch: lp:~fnordahl/ubuntu-openstack-ci/ubuntu-openstack-ci
Merge into: lp:ubuntu-openstack-ci
Diff against target: 18 lines (+8/-0)
1 file modified
run/job-parts/osci_gerrit_common.sh (+8/-0)
To merge this branch: bzr merge lp:~fnordahl/ubuntu-openstack-ci/ubuntu-openstack-ci
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Chris MacNaughton (community) Approve
Review via email: mp+378568@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

LGTM, thanks!

review: Approve
Revision history for this message
Liam Young (gnuoy) wrote :

LGTM too

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'run/job-parts/osci_gerrit_common.sh'
2--- run/job-parts/osci_gerrit_common.sh 2019-10-15 07:54:42 +0000
3+++ run/job-parts/osci_gerrit_common.sh 2020-02-05 08:28:18 +0000
4@@ -190,6 +190,14 @@
5 echo "Uploading..."
6 f_novarc_artifactcloud
7 timeout 600 swift upload $UOSCI_BUCKET_NAME --object-name $UOSCI_BLOB_PATH ${WORKSPACE}
8+ RC=$?
9+
10+ if [ $RC -eq 0 ]; then
11+ # Upload was successful, remove juju crashdump from local disk to avoid
12+ # duplicate storage and consuming all diskspace on the jenkins nodes
13+ echo "Removing local copy of juju crashdump..."
14+ rm -vf ${WORKSPACE}/juju-crashdump-*.tar.xz
15+ fi
16
17 # Set content-type for txt and html files
18 export OBJECT_MATCH_PATTERN="$UOSCI_BLOB_PATH"

Subscribers

People subscribed via source and target branches