Merge pull request #4860 from masters-of-cats/pr-process-not-found-err

crosbymichael · web-flow · commit dc207b654da0 · 2020-12-21T10:25:11.000-05:00
Return GRPC not found error instead of plain one
diff --git a/container_linux_test.go b/container_linux_test.go
@@ -1178,6 +1178,10 @@ func TestContainerLoadUnexistingProcess(t *testing.T) {
 		t.Fatal("an error should have occurred when loading a process that does not exist")
 	}
 
+	if !errdefs.IsNotFound(err) {
+		t.Fatalf("an error of type NotFound should have been returned when loading a process that does not exist, got %#v instead ", err)
+	}
+
 	if err := task.Kill(ctx, syscall.SIGKILL); err != nil {
 		t.Error(err)
 	}
diff --git a/runtime/v2/runc/v2/service.go b/runtime/v2/runc/v2/service.go
@@ -483,7 +483,7 @@ func (s *service) State(ctx context.Context, r *taskAPI.StateRequest) (*taskAPI.
 	}
 	p, err := container.Process(r.ExecID)
 	if err != nil {
-		return nil, err
+		return nil, errdefs.ToGRPC(err)
 	}
 	st, err := p.Status(ctx)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -1178,6 +1178,10 @@ func TestContainerLoadUnexistingProcess(t *testing.T) {`
`1178`	`1178`	`t.Fatal("an error should have occurred when loading a process that does not exist")`
`1179`	`1179`	`}`
`1180`	`1180`
	`1181`	`+ if !errdefs.IsNotFound(err) {`
	`1182`	`+ t.Fatalf("an error of type NotFound should have been returned when loading a process that does not exist, got %#v instead ", err)`
	`1183`	`+ }`
	`1184`	`+`
`1181`	`1185`	`if err := task.Kill(ctx, syscall.SIGKILL); err != nil {`
`1182`	`1186`	`t.Error(err)`
`1183`	`1187`	`}`
Original file line number	Diff line number	Diff line change
`@@ -483,7 +483,7 @@ func (s service) State(ctx context.Context, r taskAPI.StateRequest) (*taskAPI.`
`483`	`483`	`}`
`484`	`484`	`p, err := container.Process(r.ExecID)`
`485`	`485`	`if err != nil {`
`486`		`- return nil, err`
	`486`	`+ return nil, errdefs.ToGRPC(err)`
`487`	`487`	`}`
`488`	`488`	`st, err := p.Status(ctx)`
`489`	`489`	`if err != nil {`