diff options
| author | Jhon Honce <jhonce@redhat.com> | 2022-05-25 09:05:50 -0700 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2022-06-14 14:30:53 -0400 |
| commit | 4588a29be68afcab4a89cc2cf6859e77de22e519 (patch) | |
| tree | c3fa90c504c04025eed840ac9fb2bffe009af0d7 | |
| parent | 64fb57b416b187c6172b2e457efb19fc44c2ad1f (diff) | |
| download | podman-4588a29be68afcab4a89cc2cf6859e77de22e519.tar.gz podman-4588a29be68afcab4a89cc2cf6859e77de22e519.tar.bz2 podman-4588a29be68afcab4a89cc2cf6859e77de22e519.zip | |
Support remote deadlock errors in rm
Refactor test for deadlock by comparing error text vs. actual
ErrWillDeadlock constant. When running with --remote the error
constant will always be not equal to the error returned by the API.
```release-note
NONE
```
[NO NEW TESTS NEEDED]
Signed-off-by: Jhon Honce <jhonce@redhat.com>
| -rw-r--r-- | cmd/podman/containers/rm.go | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/cmd/podman/containers/rm.go b/cmd/podman/containers/rm.go index 420e3c38d..62143bbec 100644 --- a/cmd/podman/containers/rm.go +++ b/cmd/podman/containers/rm.go @@ -123,9 +123,7 @@ func rm(cmd *cobra.Command, args []string) error { // removeContainers will set the exit code according to the `podman-rm` man // page. func removeContainers(namesOrIDs []string, rmOptions entities.RmOptions, setExit bool) error { - var ( - errs utils.OutputErrors - ) + var errs utils.OutputErrors responses, err := registry.ContainerEngine().ContainerRm(context.Background(), namesOrIDs, rmOptions) if err != nil { if setExit { @@ -135,8 +133,9 @@ func removeContainers(namesOrIDs []string, rmOptions entities.RmOptions, setExit } for _, r := range responses { if r.Err != nil { - // TODO this will not work with the remote client - if errors.Cause(err) == define.ErrWillDeadlock { + // When using the API, errors.Cause(err) will never equal constant define.ErrWillDeadLock + if errors.Cause(r.Err) == define.ErrWillDeadlock || + errors.Cause(r.Err).Error() == define.ErrWillDeadlock.Error() { logrus.Errorf("Potential deadlock detected - please run 'podman system renumber' to resolve") } if setExit { |