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)" -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/27156 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I1cef0833c096de0fa5acf77156bb5dd362e2ef9c Gerrit-Change-Number: 27156 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange