sysdump: skip metric/gops collection if container does not exist

Do not attempt to collect metrics and gops stats if the target pod is
not running, or the specified container does not exist. This prevents
performing operations that are guaranteed to fail, and output possibly
misleading errors. One example being when the clustermesh-apiserver is
enabled, but kvstoremesh is not, as all operations targeting the latter
will fail.

Signed-off-by: Marco Iorio <[email protected]>

Author: giorio94

sysdump/sysdump.go

@@ -2413,6 +2413,11 @@ func (c *Collector) SubmitGopsSubtasks(pods []*corev1.Pod, containerName string)
 		p := p
 		for _, g := range gopsStats {
 			g := g
+
+			if !podIsRunningAndHasContainer(p, containerName) {
+				continue
+			}
+
 			if err := c.Pool.Submit(fmt.Sprintf("gops-%s-%s", p.Name, g), func(ctx context.Context) error {
 				agentPID, err := c.getGopsPID(ctx, p, containerName)
 				if err != nil {
@@ -2657,7 +2662,7 @@ func (c *Collector) submitKVStoreTasks(ctx context.Context, pod *corev1.Pod) err
 func (c *Collector) submitMetricsSubtask(pods *corev1.PodList, containerName, portName string) error {
 	for _, p := range pods.Items {
 		p := p
-		if p.Status.Phase != corev1.PodRunning {
+		if !podIsRunningAndHasContainer(&p, containerName) {
 			continue
 		}
 		err := c.Pool.Submit(fmt.Sprintf("metrics-%s-%s-%s", p.Name, containerName, portName), func(ctx context.Context) error {