fix: reverting - add cluster-health-analyzer v0.5.1 (#785) (#812) (#834)

falox · tremes · web-flow · commit 0eb65b688a2d · 2025-08-20T06:53:58.000Z
This reverts commit 315b436. Co-authored-by: Tomas Remes <tremes@users.noreply.github.com>
diff --git a/pkg/controllers/uiplugin/components.go b/pkg/controllers/uiplugin/components.go
@@ -117,7 +117,6 @@ func pluginComponentReconcilers(plugin *uiv1alpha1.UIPlugin, pluginInfo UIPlugin
 		components = append(components, reconciler.NewUpdater(newHealthAnalyzerPrometheusRole(namespace), plugin))
 		components = append(components, reconciler.NewUpdater(newHealthAnalyzerPrometheusRoleBinding(namespace), plugin))
 		components = append(components, reconciler.NewUpdater(newHealthAnalyzerService(namespace), plugin))
-		components = append(components, reconciler.NewUpdater(newHealthAnalyzerServiceMCP(namespace), plugin))
 		components = append(components, reconciler.NewUpdater(newHealthAnalyzerDeployment(namespace, serviceAccountName, pluginInfo), plugin))
 		components = append(components, reconciler.NewUpdater(newHealthAnalyzerServiceMonitor(namespace), plugin))
 	}
diff --git a/pkg/controllers/uiplugin/health_analyzer.go b/pkg/controllers/uiplugin/health_analyzer.go
@@ -94,37 +94,6 @@ func newHealthAnalyzerService(namespace string) *corev1.Service {
 	return service
 }
 
-func newHealthAnalyzerServiceMCP(namespace string) *corev1.Service {
-	service := &corev1.Service{
-		TypeMeta: metav1.TypeMeta{
-			APIVersion: corev1.SchemeGroupVersion.String(),
-			Kind:       "Service",
-		},
-		ObjectMeta: metav1.ObjectMeta{
-			Name:      name + "-mcp",
-			Namespace: namespace,
-			Annotations: map[string]string{
-				"service.beta.openshift.io/serving-cert-secret-name": volumeMountName,
-			},
-			Labels: componentLabels(name),
-		},
-		Spec: corev1.ServiceSpec{
-			Ports: []corev1.ServicePort{
-				{
-					Name:       "mcp",
-					Port:       8085,
-					TargetPort: intstr.FromString("mcp"),
-				},
-			},
-			Selector: map[string]string{
-				"app.kubernetes.io/instance": name,
-			},
-		},
-	}
-
-	return service
-}
-
 func newHealthAnalyzerDeployment(namespace string, serviceAccountName string, pluginInfo UIPluginInfo) *appsv1.Deployment {
 	deploy := &appsv1.Deployment{
 		TypeMeta: metav1.TypeMeta{
@@ -191,37 +160,6 @@ func newHealthAnalyzerDeployment(namespace string, serviceAccountName string, pl
 								},
 							},
 						},
-						{
-							Name:            name + "mcp",
-							Image:           pluginInfo.HealthAnalyzerImage,
-							ImagePullPolicy: corev1.PullAlways,
-							Args: []string{
-								"mcp",
-							},
-							Env: []corev1.EnvVar{
-								{
-									Name:  "PROM_URL",
-									Value: "https://thanos-querier.openshift-monitoring.svc.cluster.local:9091/",
-								},
-							},
-							SecurityContext: &corev1.SecurityContext{
-								RunAsNonRoot:             ptr.To(true),
-								AllowPrivilegeEscalation: ptr.To(false),
-								Capabilities: &corev1.Capabilities{
-									Drop: []corev1.Capability{"ALL"},
-								},
-								SeccompProfile: &corev1.SeccompProfile{
-									Type: corev1.SeccompProfileTypeRuntimeDefault,
-								},
-							},
-							Ports: []corev1.ContainerPort{
-								{
-									ContainerPort: 8085,
-									Name:          "mcp",
-								},
-							},
-							TerminationMessagePolicy: corev1.TerminationMessageFallbackToLogsOnError,
-						},
 					},
 					Volumes: []corev1.Volume{
 						{

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,6 @@ func pluginComponentReconcilers(plugin *uiv1alpha1.UIPlugin, pluginInfo UIPlugin`
`117`	`117`	`components = append(components, reconciler.NewUpdater(newHealthAnalyzerPrometheusRole(namespace), plugin))`
`118`	`118`	`components = append(components, reconciler.NewUpdater(newHealthAnalyzerPrometheusRoleBinding(namespace), plugin))`
`119`	`119`	`components = append(components, reconciler.NewUpdater(newHealthAnalyzerService(namespace), plugin))`
`120`		`- components = append(components, reconciler.NewUpdater(newHealthAnalyzerServiceMCP(namespace), plugin))`
`121`	`120`	`components = append(components, reconciler.NewUpdater(newHealthAnalyzerDeployment(namespace, serviceAccountName, pluginInfo), plugin))`
`122`	`121`	`components = append(components, reconciler.NewUpdater(newHealthAnalyzerServiceMonitor(namespace), plugin))`
`123`	`122`	`}`