chore: add ee helm chart changes to oss (#8799)

determined-ai · Mar 21, 2024 · 4b21505 · 4b21505
1 parent 4cb3d52
commit 4b21505
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/docs/release-notes/oidc-changed-vars.rst b/docs/release-notes/oidc-changed-vars.rst
@@ -0,0 +1,9 @@
+:orphan:
+
+**Breaking Change**
+
+-  Authentication: In the enterprise edition, in the master configuration, the
+   ``oidc.groups_claim_name`` setting that is used to set the string value of the authenticator's
+   claim name for groups has been changed to ``oidc.groups_attribute_name``. Similarly, the
+   ``oidc.display_name_claim_name`` setting that is used to set the user's display name in
+   Determined has been changed to ``oidc.display_name_attribute_name``.
diff --git a/helm/charts/determined/templates/master-config.yaml b/helm/charts/determined/templates/master-config.yaml
@@ -103,9 +103,6 @@ stringData:
       {{- if .Values.oidc.displayNameAttributeName }}
       display_name_attribute_name: {{ .Values.oidc.displayNameAttributeName }}
       {{- end }}
-      {{- if .Values.oidc.GroupsClaimName }}
-      groups_claim_name: {{ .Values.oidc.GroupsClaimName }}
-      {{- end }}
     {{- end }}
 
     {{- if .Values.scim }}

diff --git a/helm/charts/determined/values.yaml b/helm/charts/determined/values.yaml
@@ -116,7 +116,6 @@ useNodePortForMaster: false
 #   scimAuthenticationAttribute:
 #   autoProvisionUsers:
 #   groupsAttributeName:
-#   GroupsClaimName:
 #   displayNameAttributeName:
 
 # scim (EE-only) enables System for Cross-domain Identity Management (SCIM) integration, which is