Skip to content

Commit 026ebbc

Browse files
author
Matthew Fisher
committed
fix(builder): delete images on destroy
1 parent b4fd8e3 commit 026ebbc

3 files changed

Lines changed: 9 additions & 3 deletions

File tree

builder/image/conf.d/check-repos.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ gid = 0
66
mode = "0755"
77
keys = [
88
"/deis/services",
9+
"/deis/registry",
910
]
1011
reload_cmd = "/home/git/check-repos"
1112
check_cmd = "/app/bin/check {{ .src }}"

builder/image/templates/builder

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ usage() {
3232
}
3333

3434
parse-string(){
35-
# helper to avoid the single quote escape
35+
# helper to avoid the single quote escape
3636
# occurred in command substitution
3737
local args=() idx=0 IFS=' ' c
3838
for c; do printf -v args[idx++] '%s ' "$c"; done
@@ -181,5 +181,6 @@ echo
181181
# cleanup
182182
cd $REPO_DIR
183183
git gc &>/dev/null
184-
[ $JOB ] && docker rm -f $JOB &>/dev/null
185-
docker rmi -f $TMP_IMAGE &>/dev/null || true
184+
if [ -n "$JOB" ]; then
185+
docker rm -f $JOB &>/dev/null
186+
fi

builder/image/templates/check-repos

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,12 @@ cd $(dirname $0) # absolute path
1212
for repo in *.git;
1313
do
1414
reponame="${repo%.*}"
15+
appname="{{ .deis_registry_host }}:{{ .deis_registry_port }}/$reponame"
1516
if ! listcontains "{{ range $services := .deis_services }}{{ Base $services.Key }} {{ end }}" "$reponame";
1617
then
1718
rm -rf "$repo"
19+
docker images | grep $appname | awk '{ print $3 }' | xargs docker rmi -f
20+
# remove any dangling images left over from the cleanup
21+
docker images --filter "dangling=true" | awk '{ print $3 }' | xargs docker rmi -f
1822
fi
1923
done

0 commit comments

Comments
 (0)