containers
/
podman
mirror of https://github.com/containers/podman.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #11820 from jwhonce/issues/11810 

[NO TESTS NEEDED] Ignore removed containers
This commit is contained in:
OpenShift Merge Robot 2021-10-02 07:46:05 -04:00 committed by GitHub
parent 375fbb9805 c67593df12
commit b9d8c63813
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/domain/entities/container_ps.go
View File

@ -10,7 +10,7 @@ import (
"github.com/pkg/errors"
)
// Listcontainer describes a container suitable for listing
// ListContainer describes a container suitable for listing
type ListContainer struct {
// AutoRemove
AutoRemove bool
@ -18,7 +18,7 @@ type ListContainer struct {
Command []string
// Container creation time
Created time.Time
// Human readable container creation time.
// Human-readable container creation time.
CreatedAt string
// If container has exited/stopped
Exited bool
@ -65,7 +65,7 @@ type ListContainer struct {
Status string
}
// ListContainer Namespaces contains the identifiers of the container's Linux namespaces
// ListContainerNamespaces contains the identifiers of the container's Linux namespaces
type ListContainerNamespaces struct {
// Mount namespace
MNT string `json:"Mnt,omitempty"`

21
pkg/ps/ps.go
View File

@ -15,6 +15,7 @@ import (
"github.com/containers/podman/v3/pkg/domain/filters"
psdefine "github.com/containers/podman/v3/pkg/ps/define"
"github.com/containers/storage"
"github.com/containers/storage/types"
"github.com/pkg/errors"
"github.com/sirupsen/logrus"
)
@ -63,10 +64,14 @@ func GetContainerLists(runtime *libpod.Runtime, options entities.ContainerListOp
}
for _, con := range cons {
listCon, err := ListContainerBatch(runtime, con, options)
if err != nil {
switch {
case errors.Cause(err) == define.ErrNoSuchCtr:
continue
case err != nil:
return nil, err
default:
pss = append(pss, listCon)
}
pss = append(pss, listCon)
}
if options.All && options.External {
@ -89,7 +94,7 @@ func GetContainerLists(runtime *libpod.Runtime, options entities.ContainerListOp
return pss, nil
}
// GetExternalContainerLists returns list of external containers for e.g created by buildah
// GetExternalContainerLists returns list of external containers for e.g. created by buildah
func GetExternalContainerLists(runtime *libpod.Runtime) ([]entities.ListContainer, error) {
var (
pss = []entities.ListContainer{}
@ -102,15 +107,19 @@ func GetExternalContainerLists(runtime *libpod.Runtime) ([]entities.ListContaine
for _, con := range externCons {
listCon, err := ListStorageContainer(runtime, con)
if err != nil {
switch {
case errors.Cause(err) == types.ErrLoadError:
continue
case err != nil:
return nil, err
default:
pss = append(pss, listCon)
}
pss = append(pss, listCon)
}
return pss, nil
}
// BatchContainerOp is used in ps to reduce performance hits by "batching"
// ListContainerBatch is used in ps to reduce performance hits by "batching"
// locks.
func ListContainerBatch(rt *libpod.Runtime, ctr *libpod.Container, opts entities.ContainerListOptions) (entities.ListContainer, error) {
var (