feat(influxdb3-core): initial implementation#802
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a brand-new influxdb3-core Helm chart that deploys InfluxDB 3 Core as a single-node StatefulSet, with configurable object storage backends, optional TLS/auth settings, ingress, and Prometheus Operator monitoring.
Changes:
- Added initial chart scaffolding (
Chart.yaml,values.yaml,README.md) and CI values for in-memory object storage. - Implemented core Kubernetes manifests (StatefulSet, Service, ServiceAccount, ConfigMap, Secrets, Ingress, ServiceMonitor, NOTES).
- Added helpers for validation, naming, image selection, probes, and secret/env wiring.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| charts/influxdb3-core/Chart.yaml | New chart metadata (name/version/appVersion). |
| charts/influxdb3-core/values.yaml | Default values for object storage, security, ingress, monitoring, probes, and resources. |
| charts/influxdb3-core/README.md | Usage and configuration documentation for the chart. |
| charts/influxdb3-core/ci/memory-values.yaml | CI values enabling in-memory object storage. |
| charts/influxdb3-core/templates/_helpers.tpl | Template helpers: naming/labels, validation, probe rendering, secret/env helpers. |
| charts/influxdb3-core/templates/configmap.yaml | ConfigMap rendering InfluxDB3 environment variables and running validation. |
| charts/influxdb3-core/templates/statefulset.yaml | Main workload definition and volume/secret/env wiring. |
| charts/influxdb3-core/templates/service.yaml | Service exposing the HTTP port. |
| charts/influxdb3-core/templates/serviceaccount.yaml | Optional ServiceAccount creation. |
| charts/influxdb3-core/templates/secret-object-storage.yaml | Optional secrets for S3/Azure/GCS credentials and AWS credentials file. |
| charts/influxdb3-core/templates/secret-tls.yaml | Optional TLS secret creation (inline cert/key). |
| charts/influxdb3-core/templates/ingress-http.yaml | Optional HTTP ingress routes for write/query endpoints. |
| charts/influxdb3-core/templates/ingress-flight.yaml | Optional Flight/gRPC ingress routes. |
| charts/influxdb3-core/templates/servicemonitor.yaml | Optional Prometheus Operator ServiceMonitor for /metrics. |
| charts/influxdb3-core/templates/NOTES.txt | Post-install access instructions and configuration summary. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bednar
left a comment
bednar
left a comment
There was a problem hiding this comment.
Thanks for the initial chart. I found two issues that should be fixed before merge: TLS ServiceMonitor scraping and the deprecated DataFusion thread env var.
|
Thank you @alespour for the implementation 🙌 |
bednar
left a comment
bednar
left a comment
There was a problem hiding this comment.
Looks good to me. The previous requested changes are addressed, and I did not find new blocking issues in the updated review.
Adds chart for InfluxDB 3 Core. Closes #737 .