Updates comment

tony-landreth · tony-landreth · commit 92d95058bb00 · 2025-02-21T17:32:10.000-05:00
diff --git a/internal/controller/postgrescluster/pgmonitor.go b/internal/controller/postgrescluster/pgmonitor.go
@@ -157,12 +157,9 @@ func (r *Reconciler) reconcileMonitoringSecret(
 		return nil, err
 	}
 
+	// Checking if the exporter is enabled to determine when monitoring
+	// secret should be created.
 	if !pgmonitor.ExporterEnabled(ctx, cluster) && !feature.Enabled(ctx, feature.OpenTelemetryMetrics) {
-		// TODO: Checking if the exporter is enabled to determine when monitoring
-		// secret should be created. If more tools are added to the monitoring
-		// suite, they could need the secret when the exporter is not enabled.
-		// This check may need to be updated.
-		// Exporter is disabled; delete monitoring secret if it exists.
 		if err == nil {
 			err = errors.WithStack(r.deleteControlled(ctx, cluster, existing))
 		}

Original file line number	Diff line number	Diff line change
`@@ -157,12 +157,9 @@ func (r *Reconciler) reconcileMonitoringSecret(`
`157`	`157`	`return nil, err`
`158`	`158`	`}`
`159`	`159`
	`160`	`+ // Checking if the exporter is enabled to determine when monitoring`
	`161`	`+ // secret should be created.`
`160`	`162`	`if !pgmonitor.ExporterEnabled(ctx, cluster) && !feature.Enabled(ctx, feature.OpenTelemetryMetrics) {`
`161`		`- // TODO: Checking if the exporter is enabled to determine when monitoring`
`162`		`- // secret should be created. If more tools are added to the monitoring`
`163`		`- // suite, they could need the secret when the exporter is not enabled.`
`164`		`- // This check may need to be updated.`
`165`		`- // Exporter is disabled; delete monitoring secret if it exists.`
`166`	`163`	`if err == nil {`
`167`	`164`	`err = errors.WithStack(r.deleteControlled(ctx, cluster, existing))`
`168`	`165`	`}`