TykTechnologies / tyk-charts

Mozilla Public License 2.0
25 stars 21 forks source link

[TT-13445] Add opa support #350

Closed zalbiraw closed 4 days ago

zalbiraw commented 1 month ago

Description

Add ability to turn on OPA rules from the chart directly.

Related Issue

Motivation and Context

Issues with OPA and bootstrapping uncovered this. Default values can help direct the consumer to use the correct config rather than enabling things through extraEnvs

Test Coverage For This Change

Simply run a bootstrap with OPA enabled. if dashboard.opa.allowAdminPasswordReset is set to true the bootstrap will succeed if not it will fail with an error

You do not have permission to reset the password for other users. As an admin user, this permission can be modified using OPA rules.

Screenshots (if appropriate)

Types of changes

Checklist

komalsukhani commented 3 weeks ago

@zalbiraw Thank you for the contribution. As always PR looks perfect. I have left minor suggestions. Can you also fix lint errors?