From 08991ccbb443e42f3a464c3e4b9339e396fbe221 Mon Sep 17 00:00:00 2001
From: Elior Erez <eerez@redhat.com>
Date: Wed, 24 Jun 2026 19:38:48 -0400
Subject: [PATCH] OSAC-1768: Add storage controller to Helm chart

The OSAC_ENABLE_STORAGE_CONTROLLER env var was missing from the
operator Helm chart, so the Storage Controller never ran in
Helm-based deployments. Without it, Tenant.Status.StorageClasses
is never populated, causing all ComputeInstance provisioning to
fail with "no tenant_storage_classes available".

Default to false to avoid breaking existing deployments.
---
 charts/operator/templates/deployment.yaml | 2 ++
 charts/operator/values.yaml               | 1 +
 2 files changed, 3 insertions(+)

diff --git a/charts/operator/templates/deployment.yaml b/charts/operator/templates/deployment.yaml
index da26d7fc..69b7e659 100644
--- a/charts/operator/templates/deployment.yaml
+++ b/charts/operator/templates/deployment.yaml
@@ -106,6 +106,8 @@ spec:
             value: {{ .Values.controllers.networking | quote }}
           - name: OSAC_ENABLE_BAREMETAL_INSTANCE_CONTROLLER
             value: {{ .Values.controllers.bareMetalInstance | quote }}
+          - name: OSAC_ENABLE_STORAGE_CONTROLLER
+            value: {{ .Values.controllers.storageController | quote }}
         securityContext:
           allowPrivilegeEscalation: false
           readOnlyRootFilesystem: true
diff --git a/charts/operator/values.yaml b/charts/operator/values.yaml
index f38e5ab1..48df1a78 100644
--- a/charts/operator/values.yaml
+++ b/charts/operator/values.yaml
@@ -30,6 +30,7 @@ controllers:
   tenant: true
   networking: true
   bareMetalInstance: true
+  storageController: false
 
 configSecret:
   name: "osac-config"