diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2022-04-24 06:03:53 -0400 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2022-04-24 06:07:10 -0400 |
| commit | 0d83f4b768af46b95d6c5a652b2c56c2ae2093f0 (patch) | |
| tree | 74b087a2cf032240f4d3566154b5ac1ade62553d /pkg/domain | |
| parent | f65f3320e1124c94db053c1f811487920ae2a70e (diff) | |
| download | podman-0d83f4b768af46b95d6c5a652b2c56c2ae2093f0.tar.gz podman-0d83f4b768af46b95d6c5a652b2c56c2ae2093f0.tar.bz2 podman-0d83f4b768af46b95d6c5a652b2c56c2ae2093f0.zip | |
Allow filtering of "removing", it is a valid status
Do not use a list of statuses outside of libpod to validate container
statuses. Removing status was never added to the list.
Fixes: https://github.com/containers/podman/issues/13986
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/domain')
| -rw-r--r-- | pkg/domain/filters/containers.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/domain/filters/containers.go b/pkg/domain/filters/containers.go index 0557e75d6..3e5b9cad9 100644 --- a/pkg/domain/filters/containers.go +++ b/pkg/domain/filters/containers.go @@ -52,8 +52,8 @@ func GenerateContainerFilterFuncs(filter string, filterValues []string, r *libpo }, nil case "status": for _, filterValue := range filterValues { - if !util.StringInSlice(filterValue, []string{"created", "running", "paused", "stopped", "exited", "unknown"}) { - return nil, errors.Errorf("%s is not a valid status", filterValue) + if _, err := define.StringToContainerStatus(filterValue); err != nil { + return nil, err } } return func(c *libpod.Container) bool { |