Skip to content

CI: prune unused Docker images after use#4104

Merged
bk2204 merged 3 commits intomasterfrom
ci-disk-space
Apr 23, 2020
Merged

CI: prune unused Docker images after use#4104
bk2204 merged 3 commits intomasterfrom
ci-disk-space

Conversation

@bk2204
Copy link
Member

@bk2204 bk2204 commented Apr 17, 2020

At the moment, we're running out of disk space when building packages in Docker containers. This is because our CI VMs have only 14 GB of storage. To help deal with this problem, let's add an option to the run_dockers.bsh script to prune images after using them so we can free some of that disk space and let the remaining jobs complete.

bk2204 added 2 commits April 17, 2020 15:59
At the moment, we're running out of disk space when building packages in
Docker containers.  This is because our CI VMs have only 14 GB of
storage.  To help deal with this problem, let's add an option to the
run_dockers.bsh script to prune images after using them so we can free
some of that disk space and let the remaining jobs complete.
When building on any CentOS version, we end up building a lot of
additional items, and this is especially true of CentOS 6, where we must
build Ruby.  These intermediate build products run us out of disk space
in CI, so let's clean up the RPM build areas and recreate them before
building Git LFS itself.
When building on any CentOS version, we end up building a lot of
additional items, and this is especially true of CentOS 6, where we must
build Ruby.  These intermediate build products run us out of disk space
in CI, so let's remove the RPMs that we installed to build Ruby once
we're done with them.
@bk2204 bk2204 requested a review from a team April 21, 2020 17:50
@bk2204 bk2204 marked this pull request as ready for review April 21, 2020 17:50
Copy link
Member

@chrisd8088 chrisd8088 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 These all seem like reasonable cleanups; thank you!

@bk2204 bk2204 merged commit d387248 into master Apr 23, 2020
chrisd8088 added a commit to chrisd8088/git-lfs that referenced this pull request Sep 13, 2023
The --prune command-line option was added to the docker/run_dockers.bsh
script in commit 4a2e96b of PR git-lfs#4104,
and the --arch command-line options was added in commit
569b5ce of PR git-lfs#4728.  Earlier, the
REPO_HOSTNAME environment variable was removed from all the corresponding
Dockerfiles in the git-lfs/build-dockers repository in commit
git-lfs/build-dockers@d4c2fe6.

We therefore update the docker/run_dockers.bsh script now to remove
the unused REPO_HOSTNAME environment variable and document the new
--prune and --arch command-line options.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants