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
Merged at revision: 428
Proposed branch: lp:~fnordahl/ubuntu-openstack-ci/ubuntu-openstack-ci
Merge into: lp:ubuntu-openstack-ci
Diff against target: 12 lines (+1/-1)
1 file modified
run/job-parts/osci_gerrit_common.sh (+1/-1)
To merge this branch: bzr merge lp:~fnordahl/ubuntu-openstack-ci/ubuntu-openstack-ci
Reviewer Review Type Date Requested Status
Chris MacNaughton (community) Approve
Review via email: mp+378598@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

LGTM; with some digging, this path seems like a safe place to recursively remove any crashdumps, as they should be limited to this job

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 2020-02-05 08:26:43 +0000
3+++ run/job-parts/osci_gerrit_common.sh 2020-02-05 15:25:52 +0000
4@@ -196,7 +196,7 @@
5 # Upload was successful, remove juju crashdump from local disk to avoid
6 # duplicate storage and consuming all diskspace on the jenkins nodes
7 echo "Removing local copy of juju crashdump..."
8- rm -vf ${WORKSPACE}/juju-crashdump-*.tar.xz
9+ find ${WORKSPACE} -name juju-crashdump-\*.tar.xz -exec rm -vf {} \;
10 fi
11
12 # Set content-type for txt and html files

Subscribers

People subscribed via source and target branches