kube play: don't print start errors twice

Luap99 · Luap99 · commit 04e8cd1eb101 · 2025-03-05T14:50:36.000+01:00
It is very bad practise to print to stdout in our backend code without
nay real context. The exact same error message is returned to the caller
and printed in the cli frontend hwere it should be.

Therefore drop this print as it is redundant.

Signed-off-by: Paul Holzinger &lt;pholzing@redhat.com&gt;
diff --git a/pkg/domain/infra/abi/play.go b/pkg/domain/infra/abi/play.go
@@ -1143,7 +1143,6 @@ func (ic *ContainerEngine) playKubePod(ctx context.Context, podName string, podY
 		}
 		for id, err := range podStartErrors {
 			playKubePod.ContainerErrors = append(playKubePod.ContainerErrors, fmt.Errorf("starting container %s: %w", id, err).Error())
-			fmt.Println(playKubePod.ContainerErrors)
 		}
 
 		// Wait for each proxy to receive a READY message. Use a wait

Original file line number	Diff line number	Diff line change
`@@ -1143,7 +1143,6 @@ func (ic *ContainerEngine) playKubePod(ctx context.Context, podName string, podY`
`1143`	`1143`	`}`
`1144`	`1144`	`for id, err := range podStartErrors {`
`1145`	`1145`	`playKubePod.ContainerErrors = append(playKubePod.ContainerErrors, fmt.Errorf("starting container %s: %w", id, err).Error())`
`1146`		`- fmt.Println(playKubePod.ContainerErrors)`
`1147`	`1146`	`}`
`1148`	`1147`
`1149`	`1148`	`// Wait for each proxy to receive a READY message. Use a wait`