osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/27156 )
Change subject: scripts/docker-cleanup.sh: conditional img clean ......................................................................
scripts/docker-cleanup.sh: conditional img clean
Only run the simple image clean code if docuum is not running. It works well enough in most cases, but has the drawbacks that it never deletes "latest" images or images not matching "^osmocom-build", and may delete images that are still being used (OS#5447). With the other tool, all images are considered for removal, and the ones that have not been used the longest time are removed first.
Related: OS#5477, OS#5066, SYS#5827 Change-Id: I1cef0833c096de0fa5acf77156bb5dd362e2ef9c --- M scripts/docker-cleanup.sh 1 file changed, 7 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/56/27156/1
diff --git a/scripts/docker-cleanup.sh b/scripts/docker-cleanup.sh index 34a98b2..7336ed2 100755 --- a/scripts/docker-cleanup.sh +++ b/scripts/docker-cleanup.sh @@ -1,10 +1,13 @@ #!/bin/sh -x
+# simple image cleaning code in case docuum isn't running # delete all but the latest images -IMAGES=`docker image ls | grep ^osmocom-build | grep -v latest | awk -F ' ' '{print $1":"$2}'` -for f in $IMAGES; do - docker image rm $f -done +if [ -z "$(docker ps -q -f name=docuum)" ]; then + IMAGES=`docker image ls | grep ^osmocom-build | grep -v latest | awk -F ' ' '{print $1":"$2}'` + for f in $IMAGES; do + docker image rm $f + done +fi
# delete all containers where we forgot to use --rm with docker run CONTAINERS="$(docker ps -q -a -f status=exited -f status=created)"
-
osmith