diff options
| author |   Steve Baker <sbaker@redhat.com> | 2017-05-16 04:39:05 +0000 |
|---|---|---|
| committer | Steve Baker <sbaker@redhat.com> | 2017-05-17 03:46:25 +0000 |
| commit | 7962dd4d44d06169e7dd3985dd23505561e2c97d (patch) | |
| tree | 1e71d8bf2d180351f0a2c01bbd7c232f888dfd83 | |
| parent | 6241c0309eb9cb3e7f771726a740f12f28f38904 (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-x | docker/docker-puppet.py | 4 |
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. |