Merge pull request #1302 from jlarriba/prometheus_internal

openshift-merge-bot[bot] · web-flow · commit 1921c1dde14a · 2025-02-25T14:20:59.000Z
[OSPRH-13961] Remove the creation of the external Prometheus route
diff --git a/pkg/openstack/telemetry.go b/pkg/openstack/telemetry.go
@@ -205,7 +205,7 @@ func ReconcileTelemetry(ctx context.Context, instance *corev1beta1.OpenStackCont
 			return ctrlResult, nil
 		}
 		// update TLS settings with cert secret
-		instance.Spec.Telemetry.Template.MetricStorage.PrometheusTLS.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointPublic)
+		instance.Spec.Telemetry.Template.MetricStorage.PrometheusTLS.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointInternal)
 
 		// TODO: rewrite this once we have TLS on alertmanager
 		for _, alertmanagerSvc := range alertmanagerSvcs.Items {

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,7 @@ func ReconcileTelemetry(ctx context.Context, instance *corev1beta1.OpenStackCont`
`205`	`205`	`return ctrlResult, nil`
`206`	`206`	`}`
`207`	`207`	`// update TLS settings with cert secret`
`208`		`- instance.Spec.Telemetry.Template.MetricStorage.PrometheusTLS.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointPublic)`
	`208`	`+ instance.Spec.Telemetry.Template.MetricStorage.PrometheusTLS.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointInternal)`
`209`	`209`
`210`	`210`	`// TODO: rewrite this once we have TLS on alertmanager`
`211`	`211`	`for _, alertmanagerSvc := range alertmanagerSvcs.Items {`