Set Container Status Reason when OOM Killed

Signed-off-by: Mrunal Patel <mrunalp@gmail.com>

cmd/crioctl/container.go

@@ -460,6 +460,7 @@ func ContainerStatus(client pb.RuntimeServiceClient, ID string) error {
 	ftm := time.Unix(0, r.Status.FinishedAt)
 	fmt.Printf("Finished: %v\n", ftm)
 	fmt.Printf("Exit Code: %v\n", r.Status.ExitCode)
+	fmt.Printf("Reason: %v\n", r.Status.Reason)
 	if r.Status.Image != nil {
 		fmt.Printf("Image: %v\n", r.Status.Image.Image)
 	}

oci/container.go

@@ -38,10 +38,11 @@ type Container struct {
 // ContainerState represents the status of a container.
 type ContainerState struct {
 	specs.State
-	Created  time.Time `json:"created"`
+	Created   time.Time `json:"created"`
-	Started  time.Time `json:"started,omitempty"`
+	Started   time.Time `json:"started,omitempty"`
-	Finished time.Time `json:"finished,omitempty"`
+	Finished  time.Time `json:"finished,omitempty"`
-	ExitCode int32     `json:"exitCode,omitempty"`
+	ExitCode  int32     `json:"exitCode,omitempty"`
+	OOMKilled bool      `json:"oomKilled,omitempty"`
 }
 
 // NewContainer creates a container object.

oci/oci.go

@@ -552,6 +552,11 @@ func (r *Runtime) UpdateStatus(c *Container) error {
 			}
 			c.state.ExitCode = int32(statusCode)
 		}
+
+		oomFilePath := filepath.Join(c.bundlePath, "oom")
+		if _, err = os.Stat(oomFilePath); err == nil {
+			c.state.OOMKilled = true
+		}
 	}
 
 	return nil

server/container_status.go

@@ -98,6 +98,9 @@ func (s *Server) ContainerStatus(ctx context.Context, req *pb.ContainerStatusReq
 		finished := cState.Finished.UnixNano()
 		resp.Status.FinishedAt = finished
 		resp.Status.ExitCode = cState.ExitCode
+		if cState.OOMKilled {
+			resp.Status.Reason = "OOMKilled"
+		}
 	}
 
 	resp.Status.State = rStatus