opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-06-13 11:44:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

Log warn instead of error for removing nonexistant container

Browse Source 
In event of a container removal that is no longer in database, log a
warning instead of an error, as there is not any problem continuing
execution.

Resolves #4314

Signed-off-by: Tyler Ramer <tyaramer@gmail.com>
This commit is contained in:
Tyler Ramer
2019-10-25 13:59:43 -04:00
parent a01cb220c8
commit 1d00acee19
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/adapter/containers.go
View File

@ -438,9 +438,13 @@ func (r *LocalRuntime) Run(ctx context.Context, c *cliconfig.RunValues, exitCode
if c.IsSet("rm") { if c.IsSet("rm") {
if err := r.Runtime.RemoveContainer(ctx, ctr, false, true); err != nil { if err := r.Runtime.RemoveContainer(ctx, ctr, false, true); err != nil {
if errors.Cause(err) == define.ErrNoSuchCtr {
logrus.Warnf("Container %s does not exist: %v", ctr.ID(), err)
} else {
logrus.Errorf("Error removing container %s: %v", ctr.ID(), err) logrus.Errorf("Error removing container %s: %v", ctr.ID(), err)
} }
} }
}
return exitCode, nil return exitCode, nil
} }