Add Web Config File Support and Enhanced Helm Chart Features#909
Add Web Config File Support and Enhanced Helm Chart Features#909zofer1zohargo wants to merge 12 commits intoburningalchemist:masterfrom
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Hey @zofer1zohargo, thank you for your contribution! 👏 Let me go through the change set and I will get back to you. 😃 UPD: Could you please bump up the chart in Chart.yaml to |
|
Pushed the changes. Let me know if I need to issue another PR or do anything |
|
Hey @zofer1zohargo, thanks! I still need to review the PR, tbh I'd split it in two. But let me finish the review, and then we discuss. 🙌 |
|
Hi @burningalchemist, did you have a chance to review the changes? Anything that you need me to explain? |
|
Hey @zofer1zohargo, apologies, I was busy developing other features. I'd say I'm not convinced on |
Summary
This PR introduces comprehensive support for web configuration files (TLS and Basic Authentication) in the SQL Exporter Helm chart, along with dynamic configuration generation and extensive deployment examples. These enhancements make it easier to deploy SQL Exporter in production environments with proper security configurations.
Key Features
🔐 Web Config File Support
⚙️ Dynamic Configuration Generation
dynamicConfigoption to generatesql_exporter.ymlat runtime from secrets📦 Init Container Integration
helm/scripts/init.sh) for:🚀 Smart Service Configuration
📚 Comprehensive Examples
Added 7 detailed deployment examples with documentation:
Each example includes complete README with step-by-step instructions, sample Kubernetes secret YAML files, example Helm values configurations, and testing commands.
Changes Overview
Modified Components
New Components
Use Cases
This PR enables several important deployment patterns:
Backward Compatibility
✅ All changes are backward compatible:
dynamicConfig.enabledandwebConfig.enabledTesting
All examples have been tested and include: