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

Merge pull request #1017 from baude/cri_tests_sandbox

Browse source 
server/sandbox_list.go: No error on filtered list find by ID
This commit is contained in:
Daniel J Walsh 2017-10-18 16:10:11 -04:00 committed by GitHub
commit c2b7f37bd3
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
server/sandbox_list.go
View file

@ -42,7 +42,11 @@ func (s *Server) ListPodSandbox(ctx context.Context, req *pb.ListPodSandboxReque
if filter.Id != "" { if filter.Id != "" {
id, err := s.PodIDIndex().Get(filter.Id) id, err := s.PodIDIndex().Get(filter.Id)
if err != nil { if err != nil {
return nil, err // Not finding an ID in a filtered list should not be considered
// and error; it might have been deleted when stop was done.
// Log and return an empty struct.
logrus.Warn("unable to find pod %s with filter", filter.Id)
return &pb.ListPodSandboxResponse{}, nil
} }
sb := s.getSandbox(id) sb := s.getSandbox(id)
if sb == nil { if sb == nil {