Fix handling of array for groups in chart

StackEng · Jul 19, 2024 · 81db36e · 81db36e
1 parent 5c1bcbb
commit 81db36e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/charts/opserver/Chart.yaml b/charts/opserver/Chart.yaml
@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 1.0.8
+version: 1.0.9
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

diff --git a/charts/opserver/templates/deployment.yaml b/charts/opserver/templates/deployment.yaml
@@ -118,10 +118,14 @@ spec:
           value: "true"
         - name: Security__Name
           value: "Okta"
-        - name: Security__ViewEverythingGroups
-          value: {{ .Values.opserverSettings.security.viewGroups | quote }}
-        - name: Security__AdminEverythingGroups
-          value: {{ .Values.opserverSettings.security.adminGroups | quote }}
+        {{- range $i, $groupName := .Values.security.viewGroups }}          
+        - name: Security__ViewEverythingGroups__{{ $i }}
+          value: {{ $groupName | quote }}
+        {{- end }}
+        {{- range $i, $groupName := .Values.security.adminGroups }}          
+        - name: Security__AdminEverythingGroups__{{ $i }}
+          value: {{ $groupName | quote }}
+        {{- end }}
         - name: Security__ClientId
           valueFrom:
             secretKeyRef: