-
Notifications
You must be signed in to change notification settings - Fork 22
Conversation
Signed-off-by: Faisal Memon <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a comment in the generated notes like:
https://github.com/spiffe/helm-charts/blob/main/charts/spire/templates/NOTES.txt#L6-L7
stating something like warning, experimental flags detected?
Signed-off-by: Faisal Memon <[email protected]>
Signed-off-by: Faisal Memon <[email protected]>
Signed-off-by: Faisal Memon <[email protected]>
warning added |
Signed-off-by: Faisal Memon <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add support for the experimental section of the config. Needed for developers. This PR doesnt add support for the
auth_opa_policy_engine
experimental config.