Merge pull request containers#17821 from umohnani8/detach

Add service ctr cleanup to PlayKubeDown
vrothberg · Mar 21, 2023 · d8265f0 · d8265f0
2 parents 23d97fc + 3e405a2
commit d8265f0
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/pkg/domain/infra/abi/play.go b/pkg/domain/infra/abi/play.go
@@ -1366,6 +1366,15 @@ func (ic *ContainerEngine) PlayKubeDown(ctx context.Context, body io.Reader, opt
 		}
 	}
 
+	// Remove the service container to ensure it is removed before we return for the remote case
+	// Needed for the clean up with podman kube play --wait in the remote case
+	if reports.ServiceContainerID != "" {
+		_, err = ic.ContainerRm(ctx, []string{reports.ServiceContainerID}, entities.RmOptions{})
+		if err != nil && !errors.Is(err, define.ErrNoSuchCtr) {
+			return nil, err
+		}
+	}
+
 	return reports, nil
 }
 

diff --git a/test/system/700-play.bats b/test/system/700-play.bats
@@ -619,8 +619,8 @@ spec:
     - name: server
       image: $IMAGE
       command:
-      - sleep
-      - "5"
+      - echo
+      - "hello"
 " > $fname
 
     run_podman kube play --wait $fname