dennismdejong opened a new pull request, #11668: URL: https://github.com/apache/gravitino/pull/11668
### What changes were proposed in this pull request? Add 5 missing OAuth configuration properties to the Helm chart: - `gravitino.authenticator.oauth.principalMapper` - `gravitino.authenticator.oauth.principalMapper.regex.pattern` - `gravitino.authenticator.oauth.groupsFields` - `gravitino.authenticator.oauth.groupMapper` - `gravitino.authenticator.oauth.groupMapper.regex.pattern` These properties were added to the Gravitino server in v1.2.0 and v1.3.0 but were never wired into the Helm chart template. ### Why are the changes needed? Users currently must use the `additionalConfigItems` escape hatch to configure group and principal mapping, which is error-prone and not discoverable. Adding native support makes the Helm chart fully feature-complete with the server-side OAuth configuration capabilities. Fix: #11667 ### Does this PR introduce any user-facing change? Yes, adds 5 new configurable values under `authenticator.oauth` in the Helm chart: - `principalMapper` (default: `regex`) - `principalMapperRegexPattern` (default: `^(.*)$`) - `groupsFields` (default: `groups`) - `groupMapper` (default: `regex`) - `groupMapperRegexPattern` (default: `^(.*)$`) ### How was this patch tested? Ran `helm unittest --strict dev/charts/gravitino/` — all 30 tests pass, including new asserts for the added properties. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
