github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-07-21 18:53:30 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #4549 from liubin/fix/4419-set-status-if-wait-process-failed

Browse Source 
shim: set a non-zero return code if the wait process call failed.
This commit is contained in:
GabyCT 2022-06-29 17:04:53 -05:00 committed by GitHub
commit 2a94261df5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/runtime/pkg/containerd-shim-v2/wait.go
View File

@ -53,6 +53,11 @@ func wait(ctx context.Context, s *service, c *container, execID string) (int32,
"container": c.id, "container": c.id,
"pid": processID, "pid": processID,
}).Error("Wait for process failed") }).Error("Wait for process failed")
// set return code if wait failed
if ret == 0 {
ret = exitCode255
}
} }
timeStamp := time.Now() timeStamp := time.Now()

2
src/runtime/virtcontainers/sandbox.go
View File

@ -1594,7 +1594,7 @@ func (s *Sandbox) ResumeContainer(ctx context.Context, containerID string) error
} }
// createContainers registers all containers, create the // createContainers registers all containers, create the
// containers in the guest and starts one shim per container. // containers in the guest.
func (s *Sandbox) createContainers(ctx context.Context) error { func (s *Sandbox) createContainers(ctx context.Context) error {
span, ctx := katatrace.Trace(ctx, s.Logger(), "createContainers", sandboxTracingTags, map[string]string{"sandbox_id": s.id}) span, ctx := katatrace.Trace(ctx, s.Logger(), "createContainers", sandboxTracingTags, map[string]string{"sandbox_id": s.id})
defer span.End() defer span.End()