feat: Install goldpinger daemonset if one does not exist when running goldpinger collector (#1619)

* feat: Install goldpinger if one does not exist when running goldpinger collector

- Deploy golpinger daemonset if one is not detected in the cluster
- Clean up all deployed resources
- Add delay to allow users to wait for goldpinger to perform checks

Signed-off-by: Evans Mungai <[email protected]>

* Add missing test data file

Signed-off-by: Evans Mungai <[email protected]>

* Better naming of create resource functions

Signed-off-by: Evans Mungai <[email protected]>

---------

Signed-off-by: Evans Mungai <[email protected]>

Author: banjoh

config/crds/troubleshoot.sh_collectors.yaml

@@ -348,6 +348,8 @@ spec:
                       type: object
                     goldpinger:
                       properties:
+                        collectDelay:
+                          type: string
                         collectorName:
                           type: string
                         exclude:
@@ -374,6 +376,8 @@ spec:
                             serviceAccountName:
                               type: string
                           type: object
+                        serviceAccountName:
+                          type: string
                       type: object
                     helm:
                       properties:

config/crds/troubleshoot.sh_preflights.yaml

@@ -2077,6 +2077,8 @@ spec:
                       type: object
                     goldpinger:
                       properties:
+                        collectDelay:
+                          type: string
                         collectorName:
                           type: string
                         exclude:
@@ -2103,6 +2105,8 @@ spec:
                             serviceAccountName:
                               type: string
                           type: object
+                        serviceAccountName:
+                          type: string
                       type: object
                     helm:
                       properties:

config/crds/troubleshoot.sh_supportbundles.yaml

@@ -2108,6 +2108,8 @@ spec:
                       type: object
                     goldpinger:
                       properties:
+                        collectDelay:
+                          type: string
                         collectorName:
                           type: string
                         exclude:
@@ -2134,6 +2136,8 @@ spec:
                             serviceAccountName:
                               type: string
                           type: object
+                        serviceAccountName:
+                          type: string
                       type: object
                     helm:
                       properties:

pkg/analyze/goldpinger_test.go

@@ -96,6 +96,18 @@ func TestAnalyzeGoldpinger_podPingsAnalysis(t *testing.T) {
 				},
 			},
 		},
+		{
+			name: "no ping errors for one pod",
+			cao:  caoFixture(t, "goldpinger/checkall-one-pod.json"),
+			want: []*AnalyzeResult{
+				{
+					Title:   "Pings to \"gp-goldpinger-xn9rg\" pod succeeded",
+					Message: "Pings to \"gp-goldpinger-xn9rg\" pod from all other pods in the cluster succeeded",
+					IconKey: "kubernetes",
+					IsPass:  true,
+				},
+			},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

pkg/apis/troubleshoot/v1beta2/collector_shared.go

@@ -277,9 +277,11 @@ type Helm struct {
 }
 
 type Goldpinger struct {
-	CollectorMeta    `json:",inline" yaml:",inline"`
-	Namespace        string            `json:"namespace,omitempty" yaml:"namespace,omitempty"`
-	PodLaunchOptions *PodLaunchOptions `json:"podLaunchOptions,omitempty" yaml:"podLaunchOptions,omitempty"`
+	CollectorMeta      `json:",inline" yaml:",inline"`
+	Namespace          string            `json:"namespace,omitempty" yaml:"namespace,omitempty"`
+	ServiceAccountName string            `json:"serviceAccountName,omitempty" yaml:"serviceAccountName,omitempty"`
+	CollectDelay       string            `json:"collectDelay,omitempty" yaml:"collectDelay,omitempty"`
+	PodLaunchOptions   *PodLaunchOptions `json:"podLaunchOptions,omitempty" yaml:"podLaunchOptions,omitempty"`
 }
 
 type PodLaunchOptions struct {