vbatts/cri-o
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Specifying a filter with no filtering expressions is now idempotent

Browse source 
Signed-off-by: Yann Ramin <atrus@stackworks.net>
This commit is contained in:
Yann Ramin 2018-01-31 10:28:01 -08:00
parent 14c1c70407
commit 50c94a9335
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
server/container_list.go
View file

@ -30,7 +30,9 @@ func filterContainer(c *pb.Container, filter *pb.ContainerFilter) bool {
// filterContainerList applies a protobuf-defined filter to retrieve only intended containers. Not matching
// the filter is not considered an error but will return an empty response.
func (s *Server) filterContainerList(filter *pb.ContainerFilter) (ctrList []*oci.Container, err error) {
func (s *Server) filterContainerList(filter *pb.ContainerFilter, origCtrList []*oci.Container) (ctrList []*oci.Container, err error) {
ctrList = origCtrList
// Filter using container id and pod id first.
if filter.Id != "" {
id, err := s.CtrIDIndex().Get(filter.Id)
@ -83,7 +85,7 @@ func (s *Server) ListContainers(ctx context.Context, req *pb.ListContainersReque
}
if filter != nil {
ctrList, err = s.filterContainerList(filter)
ctrList, err = s.filterContainerList(filter, ctrList)
if err != nil {
return nil, err
}

2
server/container_stats_list.go
View file

@ -28,7 +28,7 @@ func (s *Server) ListContainerStats(ctx context.Context, req *pb.ListContainerSt
PodSandboxId: req.Filter.PodSandboxId,
LabelSelector: req.Filter.LabelSelector,
}
ctrList, err = s.filterContainerList(cFilter)
ctrList, err = s.filterContainerList(cFilter, ctrList)
if err != nil {
return nil, err
}