aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteve Baker <sbaker@redhat.com>2017-05-16 04:39:05 +0000
committerSteve Baker <sbaker@redhat.com>2017-05-17 03:46:25 +0000
commit7962dd4d44d06169e7dd3985dd23505561e2c97d (patch)
tree1e71d8bf2d180351f0a2c01bbd7c232f888dfd83
parent6241c0309eb9cb3e7f771726a740f12f28f38904 (diff)
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
-rwxr-xr-xdocker/docker-puppet.py4
1 files changed, 3 insertions, 1 deletions
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.