Merge lp:~robru/cupstream2distro/delete-distro-versions-after-diffing into lp:cupstream2distro

Proposed by Robert Bruce Park on 2015-02-16
Status: Merged
Approved by: Robert Bruce Park on 2015-02-16
Approved revision: 873
Merged at revision: 873
Proposed branch: lp:~robru/cupstream2distro/delete-distro-versions-after-diffing
Merge into: lp:cupstream2distro
Diff against target: 14 lines (+4/-0)
1 file modified
citrain/jenkins-templates/build.xml.tmpl (+4/-0)
To merge this branch: bzr merge lp:~robru/cupstream2distro/delete-distro-versions-after-diffing
Reviewer Review Type Date Requested Status
Łukasz Zemczak 2015-02-16 Approve on 2015-02-16
PS Jenkins bot continuous-integration Approve on 2015-02-16
Review via email: mp+249864@code.launchpad.net

Commit Message

Delete distro versions after diffing.

Description of the Change

We recently had a disk space crisis where Timo's qt silo was taking up 11GB (and total space of all silos was only 13GB). In addition to a runaway backup process, we managed to fill the entire 30GB disk.

One of the features of the train is diff generation, but in order to generate the diff it needs to have two copies of the package around: the copy we generated and the version in distro. This of course doubles our disk space usage.

This branch deletes distro versions after the diff has been generated, which is harmless. It is already the case that this directory gets deleted and new distro versions are downloaded each time a new diff is generated. So the only change here is that distro versions aren't kept around forever, taking up huge amounts of disk space.

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:873
http://jenkins.qa.ubuntu.com/job/cu2d-choo-choo-ci/453/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/cu2d-choo-choo-ci/453/rebuild

review: Approve (continuous-integration)
Łukasz Zemczak (sil2100) wrote :

Looks safe, from what I see it's indeed not needed afterwards.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'citrain/jenkins-templates/build.xml.tmpl'
2--- citrain/jenkins-templates/build.xml.tmpl 2015-02-13 22:40:39 +0000
3+++ citrain/jenkins-templates/build.xml.tmpl 2015-02-16 17:46:50 +0000
4@@ -101,6 +101,10 @@
5
6 cp *.diff "$WORKSPACE" 2>/dev/null || true
7
8+# ubuntu/ directory only contains distro versions, and only used for diffing.
9+# This cuts our disk usage in half:
10+rm -rf ubuntu/
11+
12 exit $RETVAL
13 </command>
14 </hudson.tasks.Shell>

Subscribers

People subscribed via source and target branches