From 77fe2975eac3c95d893ca22a6d0fb1f94e1b4b50 Mon Sep 17 00:00:00 2001
From: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
Date: Fri, 7 Feb 2025 17:01:28 -0500
Subject: [PATCH] incorporating code review comments.

Signed-off-by: lrangine <19699092+lokeshrangineni@users.noreply.github.com>
---
 .../v1alpha1_featurestore_postgres_db_volumes_ssl.yaml    | 8 ++++----
 .../internal/controller/services/services.go              | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/infra/feast-operator/config/samples/v1alpha1_featurestore_postgres_db_volumes_ssl.yaml b/infra/feast-operator/config/samples/v1alpha1_featurestore_postgres_db_volumes_ssl.yaml
index ad5f617887..bfef906aab 100644
--- a/infra/feast-operator/config/samples/v1alpha1_featurestore_postgres_db_volumes_ssl.yaml
+++ b/infra/feast-operator/config/samples/v1alpha1_featurestore_postgres_db_volumes_ssl.yaml
@@ -17,7 +17,7 @@ metadata:
   namespace: default
 stringData:
   sql: |
-    path: postgresql+psycopg://admin:password@postgresql.default.svc.cluster.local:5432/mydatabase?sslmode=require&sslrootcert=/var/lib/postgresql/certs/ca.crt&sslcert=/var/lib/postgresql/certs/tls.crt&sslkey=/var/lib/postgresql/certs/tls.key
+    path: postgresql+psycopg://${POSTGRES_USER}:${POSTGRES_PASSWORD}@postgresql.default.svc.cluster.local:5432/${POSTGRES_DB}?sslmode=require&sslrootcert=/var/lib/postgresql/certs/ca.crt&sslcert=/var/lib/postgresql/certs/tls.crt&sslkey=/var/lib/postgresql/certs/tls.key
     cache_ttl_seconds: 60
     sqlalchemy_config_kwargs:
         echo: false
@@ -25,10 +25,10 @@ stringData:
   postgres: |
     host: postgresql.default.svc.cluster.local
     port: 5432
-    database: mydatabase
+    database: ${POSTGRES_DB}
     db_schema: public
-    user: admin
-    password: password
+    user: ${POSTGRES_USER}
+    password: ${POSTGRES_PASSWORD}
     sslmode: require
     sslkey_path: /var/lib/postgresql/certs/tls.key
     sslcert_path: /var/lib/postgresql/certs/tls.crt
diff --git a/infra/feast-operator/internal/controller/services/services.go b/infra/feast-operator/internal/controller/services/services.go
index c4d13f40ec..ece4307459 100644
--- a/infra/feast-operator/internal/controller/services/services.go
+++ b/infra/feast-operator/internal/controller/services/services.go
@@ -455,7 +455,7 @@ func (feast *FeastServices) getVolumeMounts(feastType FeastServiceType) (volumeM
 			return appliedServices.OnlineStore.Server.VolumeMounts
 		}
 	case RegistryFeastType:
-		if feast.isLocalRegistry() {
+		if feast.isRegistryServer() {
 			return appliedServices.Registry.Local.Server.VolumeMounts
 		}
 	case UIFeastType: