Don't cache Docker layers
Merge request reports
Activity
requested review from @jbrost
added 7 commits
-
8c5bdbf5...a731206f - 6 commits from branch
master
- acee9ef5 - Don't cache Docker layers
-
8c5bdbf5...a731206f - 6 commits from branch
added 3 commits
-
acee9ef5...afab39f5 - 2 commits from branch
master
- ec623cc4 - Don't cache Docker layers
-
acee9ef5...afab39f5 - 2 commits from branch
Reason for the change: There are commands in the Dockerfiles that (intentionally) produce different results when they are run again:
-
apt-get upgrade -y
: If you rerun the pipeline, you expect to rebuild containers with all the new updates available -
git clone https://git.icinga.com/build-docker/scripts.git /usr/local/bin
: better have an up-to-date version. Untilicinga-provide-go
was added, this wasn't to important as the entrypoint pulls this repo, but now that script might be updated to install a newer Go version so you want to install it when that script is called when the image is built. This is also the concrete reason why this change is done now, otherwise the containers aren't rebuilt with Go 1.18 installed.
(This should have been in the PR description. Just look at 28425960 now: no motivation why the change was done, I can only guess and I presume that was done because it's nice for local testing because things are faster then.)
-
mentioned in commit b837d10c
mentioned in merge request debian!6 (merged)
Please register or sign in to reply