bugfix: not remove the metadata when remove container failed

Signed-off-by: Starnop <starnop@163.com>
AliyunContainerService · Oct 23, 2018 · a0376b3 · a0376b3
1 parent 53bd896
commit a0376b3
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/cri/v1alpha2/cri.go b/cri/v1alpha2/cri.go
@@ -271,6 +271,7 @@ func (c *CriManager) RunPodSandbox(ctx context.Context, r *runtime.RunPodSandbox
 		if retErr != nil {
 			if err := c.ContainerMgr.Remove(ctx, id, &apitypes.ContainerRemoveOptions{Volumes: true, Force: true}); err != nil {
 				logrus.Errorf("failed to remove container when running sandbox failed %q: %v", id, err)
+				return
 			}
 			// should not remove the sandbox container metadata from sandboxStore
 			// until it was removed by pouchd.