From 7962dd4d44d06169e7dd3985dd23505561e2c97d Mon Sep 17 00:00:00 2001 From: Steve Baker Date: Tue, 16 May 2017 04:39:05 +0000 Subject: Don't delete failed docker-puppet containers This helps a bit with debugging issues, and the container will be deleted on the next run when the same volume is configured. Change-Id: I4f2f219bd7e40abafd0eb31c1275fdd8ed4db4da --- docker/docker-puppet.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'docker/docker-puppet.py') diff --git a/docker/docker-puppet.py b/docker/docker-puppet.py index f1594d2d..49dd00cd 100755 --- a/docker/docker-puppet.py +++ b/docker/docker-puppet.py @@ -253,7 +253,9 @@ def mp_puppet_config((config_volume, puppet_tags, manifest, config_image, volume log.debug(cmd_stderr) if subproc.returncode != 0: log.error('Failed running docker-puppet.py for %s' % config_volume) - rm_container('docker-puppet-%s' % config_volume) + else: + # only delete successful runs, for debugging + rm_container('docker-puppet-%s' % config_volume) return subproc.returncode # Holds all the information for each process to consume. -- cgit 1.2.3-korg