-
Notifications
You must be signed in to change notification settings - Fork 98
docs: 3453/helm env updates #3679
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Updated all Helm value names to use consolidated chart structure: - Backend: config.backend.* prefix - Auth: config.auth.* prefix - Frontend: config.frontend.* prefix Key changes: - Fixed default values (USE_TIGERBEETLE=false, TRUST_PROXY=true) - Added secret reference support for sensitive values - Updated port numbers and configuration paths - Added missing variables from codebase - Corrected Helm value mappings to match actual chart
- Update of Helm variable names based on updated Helm chart
✅ Deploy Preview for brilliant-pasca-3e80ec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🚀 Performance Test ResultsTest Configuration:
Test Metrics:
📜 Logs
|
I can't speak to the veracity, but otherwise it looks good :) I did want to call out that you changed some tables to list the variables in alphabetical order, while others stayed the way they are. You can leave them as is, but I wanted to point that out in case alphabetizing all of them was your intent. |
Addresses #3453
Map Helm variable names to the updated Helm chart in https://github.com/interledger/charts/tree/main/charts/rafiki/values.yaml
Required
Conditional