diff options
author | Daniel J Walsh <dwalsh@redhat.com> | 2021-04-12 14:43:58 -0400 |
---|---|---|
committer | Daniel J Walsh <dwalsh@redhat.com> | 2021-04-12 14:54:37 -0400 |
commit | 801acb01e227c251e534efa12eb33a2eda4df2de (patch) | |
tree | b24eb4d6ca9637085f9a02aaa21938dd479fe1dc /pkg/namespaces | |
parent | 7203e0a76a82e278b6a41e7a5851463a0b2f571e (diff) | |
download | podman-801acb01e227c251e534efa12eb33a2eda4df2de.tar.gz podman-801acb01e227c251e534efa12eb33a2eda4df2de.tar.bz2 podman-801acb01e227c251e534efa12eb33a2eda4df2de.zip |
Do not delete container twice
10 lines above we had
// Set ContainerStateRemoving
c.state.State = define.ContainerStateRemoving
Which causes the state to not be the two checked states. Since the
c.cleanup call already deleted the OCI state, this meant that we were
calling cleanup, and hence the postHook hook twice.
Fixes: https://github.com/containers/podman/issues/9983
[NO TESTS NEEDED] Since it would be difficult to tests this. Main tests
should handle that the container is being deleted successfully.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/namespaces')
0 files changed, 0 insertions, 0 deletions