Fix: handle external database case for postgres exporter setup (#138)

Author: Tvion

controllers/postgresservice_controller.go

@@ -399,15 +399,20 @@ func (r *PostgresServiceReconciler) reconcilePostgresServiceCluster(cr *qubershi
 
 	// configure postgres-exporter user
 	if cr.Spec.PostgresExporter != nil && cr.Spec.PostgresExporter.Install {
-		patroniCore, err := r.helper.GetPatroniCoreCR()
-		if err != nil {
-			r.logger.Error("Can't get PatroniCore CR")
-			panic(err)
+		skipInstall := false
+		if cr.Spec.ExternalDataBase == nil {
+			patroniCore, err := r.helper.GetPatroniCoreCR()
+			if err != nil {
+				r.logger.Error("Can't get PatroniCore CR")
+				panic(err)
+			}
+			if patroniCore.Spec.Patroni.StandbyCluster != nil {
+				r.logger.Info("PatroniCore indicates standby cluster; skipping postgres-exporter setup (read-only)")
+				skipInstall = true
+			}
 		}
-		if patroniCore.Spec.Patroni.StandbyCluster != nil {
-			r.logger.Info("PatroniCore indicates standby cluster; skipping postgres-exporter setup (read-only)")
-		} else {
-			if err := postgresexporter.SetUpExporter(cr.Spec.PostgresExporter); err != nil { //REWORK
+		if !skipInstall {
+			if err := postgresexporter.SetUpExporter(cr.Spec.PostgresExporter); err != nil {
 				return err
 			}
 		}