New icons for analyzers

Author: divolgin

pkg/analyze/analyzer.go

@@ -13,6 +13,7 @@ type AnalyzeResult struct {
 	Title   string
 	Message string
 	URI     string
+	IconKey string
 }
 
 type getCollectedFileContents func(string) ([]byte, error)

pkg/analyze/cluster_version.go

@@ -30,7 +30,6 @@ func analyzeClusterVersion(analyzer *troubleshootv1beta1.ClusterVersion, getColl
 }
 
 func analyzeClusterVersionResult(k8sVersion semver.Version, outcomes []*troubleshootv1beta1.Outcome, checkName string) (*AnalyzeResult, error) {
-	result := AnalyzeResult{}
 	for _, outcome := range outcomes {
 		when := ""
 		message := ""
@@ -41,8 +40,9 @@ func analyzeClusterVersionResult(k8sVersion semver.Version, outcomes []*troubles
 			title = "Required Kubernetes Version"
 		}
 
-		result = AnalyzeResult{
-			Title: title,
+		result := AnalyzeResult{
+			Title:   title,
+			IconKey: "kubernetes_cluster_version",
 		}
 
 		if outcome.Fail != nil {

pkg/analyze/common_status.go

@@ -8,9 +8,10 @@ import (
 	troubleshootv1beta1 "github.com/replicatedhq/troubleshoot/pkg/apis/troubleshoot/v1beta1"
 )
 
-func commonStatus(outcomes []*troubleshootv1beta1.Outcome, title string, readyReplicas int) (*AnalyzeResult, error) {
+func commonStatus(outcomes []*troubleshootv1beta1.Outcome, title, iconKey string, readyReplicas int) (*AnalyzeResult, error) {
 	result := &AnalyzeResult{
-		Title: title,
+		Title:   title,
+		IconKey: iconKey,
 	}
 
 	// ordering from the spec is important, the first one that matches returns

pkg/analyze/container_runtime.go

@@ -27,7 +27,8 @@ func analyzeContainerRuntime(analyzer *troubleshootv1beta1.ContainerRuntime, get
 	}
 
 	result := &AnalyzeResult{
-		Title: "Container Runtime",
+		Title:   "Container Runtime",
+		IconKey: "kubernetes_container_runtime",
 	}
 
 	// ordering is important for passthrough

pkg/analyze/crd.go

@@ -25,7 +25,8 @@ func analyzeCustomResourceDefinition(analyzer *troubleshootv1beta1.CustomResourc
 	}
 
 	result := AnalyzeResult{
-		Title: title,
+		Title:   title,
+		IconKey: "kubernetes_custom_resource_definition",
 	}
 
 	for _, storageClass := range crds {

pkg/analyze/deployment_status.go

@@ -32,10 +32,11 @@ func analyzeDeploymentStatus(analyzer *troubleshootv1beta1.DeploymentStatus, get
 		// there's not an error, but maybe the requested deployment is not even deployed
 		return &AnalyzeResult{
 			Title:   fmt.Sprintf("%s Deployment Status", analyzer.Name),
+			IconKey: "kubernetes_deployment_status",
 			IsFail:  true,
 			Message: fmt.Sprintf("The deployment %q was not found", analyzer.Name),
 		}, nil
 	}
 
-	return commonStatus(analyzer.Outcomes, fmt.Sprintf("%s Status", analyzer.Name), int(status.ReadyReplicas))
+	return commonStatus(analyzer.Outcomes, fmt.Sprintf("%s Status", analyzer.Name), "kubernetes_deployment_status", int(status.ReadyReplicas))
 }

pkg/analyze/distribution.go

@@ -95,7 +95,8 @@ func analyzeDistribution(analyzer *troubleshootv1beta1.Distribution, getCollecte
 	}
 
 	result := &AnalyzeResult{
-		Title: "Kubernetes Distribution",
+		Title:   "Kubernetes Distribution",
+		IconKey: "kubernetes_distribution",
 	}
 
 	// ordering is important for passthrough

pkg/analyze/image_pull_secret.go

@@ -25,6 +25,7 @@ func analyzeImagePullSecret(analyzer *troubleshootv1beta1.ImagePullSecret, getCh
 
 	result := AnalyzeResult{
 		Title:   analyzer.CheckName,
+		IconKey: "kubernetes_image_pull_secret",
 		IsFail:  true,
 		Message: failOutcome.Fail.Message,
 		URI:     failOutcome.Fail.URI,

pkg/analyze/ingress.go

@@ -25,7 +25,8 @@ func analyzeIngress(analyzer *troubleshootv1beta1.Ingress, getCollectedFileConte
 	}
 
 	result := AnalyzeResult{
-		Title: title,
+		Title:   title,
+		IconKey: "kubernetes_ingress",
 	}
 
 	for _, ingress := range ingresses {

pkg/analyze/node_resources.go

@@ -43,7 +43,8 @@ func analyzeNodeResources(analyzer *troubleshootv1beta1.NodeResources, getCollec
 	}
 
 	result := &AnalyzeResult{
-		Title: title,
+		Title:   title,
+		IconKey: "kubernetes_node_resources",
 	}
 
 	for _, outcome := range analyzer.Outcomes {