diff options
| author | baude <bbaude@redhat.com> | 2018-12-11 16:10:35 -0600 |
|---|---|---|
| committer | baude <bbaude@redhat.com> | 2018-12-12 10:55:53 -0600 |
| commit | 9786542620afea96d5eee2d05b7e1dec38a8235d (patch) | |
| tree | 092e40048677955efe9a1e0d99c69c332d153726 /cmd/podman/run.go | |
| parent | 8645df84dbb21f5988b46e6646bb0dd04fdb2b51 (diff) | |
| download | podman-9786542620afea96d5eee2d05b7e1dec38a8235d.tar.gz podman-9786542620afea96d5eee2d05b7e1dec38a8235d.tar.bz2 podman-9786542620afea96d5eee2d05b7e1dec38a8235d.zip | |
failed containers with --rm should remove themselves
when starting or running a container that has --rm, if the starting
container fails (like due to an invalid command), the container should
get removed.
Resolves: #1985
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'cmd/podman/run.go')
| -rw-r--r-- | cmd/podman/run.go | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/cmd/podman/run.go b/cmd/podman/run.go index a4b5c918e..20cb85347 100644 --- a/cmd/podman/run.go +++ b/cmd/podman/run.go @@ -116,6 +116,11 @@ func runCmd(c *cli.Context) error { if strings.Index(err.Error(), "permission denied") > -1 { exitCode = 126 } + if c.IsSet("rm") { + if deleteError := runtime.RemoveContainer(ctx, ctr, true); deleteError != nil { + logrus.Errorf("unable to remove container %s after failing to start and attach to it", ctr.ID()) + } + } return err } |