Skip to content

fix(influxdb3-ent): add gRPC/Flight Ingress route#769

Open
alespour wants to merge 4 commits intomasterfrom
fix/enterprise-ingress-flight
Open

fix(influxdb3-ent): add gRPC/Flight Ingress route#769
alespour wants to merge 4 commits intomasterfrom
fix/enterprise-ingress-flight

Conversation

@alespour
Copy link
Copy Markdown
Contributor

@alespour alespour commented Mar 5, 2026

This PR

  • adds gRPC/Flight Ingress routing to InfluxDB3 Enterprise chart. It is required for querying with v3 client libraries.
  • keeps a single default ingress host for path-based write/query/Flight routing; an optional ingress.flight.host override is available for ingress-controller environments that require host-level separation of gRPC and HTTP traffic.

@alespour alespour marked this pull request as ready for review March 5, 2026 14:02
@alespour alespour requested a review from Copilot March 5, 2026 14:02
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a dedicated Ingress resource for Apache Arrow Flight/gRPC query traffic to InfluxDB 3 Enterprise queriers, enabling v3 client libraries to query successfully via Flight endpoints.

Changes:

  • Add ingress.flight configuration (host/TLS override, gRPC paths, and annotations) to values.yaml.
  • Introduce a new Helm template to create a *-flight Ingress routing Flight/gRPC paths to the querier service.
  • Bump the chart version to 0.2.0.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
charts/influxdb3-enterprise/values.yaml Adds configurable values for the new Flight/gRPC ingress (paths, annotations, optional host/TLS overrides).
charts/influxdb3-enterprise/templates/ingress-flight.yaml New Ingress template to route /arrow.flight.protocol.* paths to queriers using gRPC backend protocol.
charts/influxdb3-enterprise/Chart.yaml Chart version bump to reflect the new feature.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread charts/influxdb3-enterprise/templates/ingress-flight.yaml
Comment thread charts/influxdb3-enterprise/values.yaml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alespour alespour changed the title fix: add missing InfluxDB3 Enterprise gRPC/Flight Ingress route fix(influxdb3-ent): add missing gRPC/Flight Ingress route Mar 5, 2026
@alespour alespour changed the title fix(influxdb3-ent): add missing gRPC/Flight Ingress route fix(influxdb3-ent): add gRPC/Flight Ingress route Mar 5, 2026
@alespour alespour requested review from bednar and srebhan March 5, 2026 14:23
@alespour alespour removed request for bednar and srebhan March 31, 2026 16:21
@alespour alespour force-pushed the fix/enterprise-ingress-flight branch from 48c24e1 to 2b87dff Compare March 31, 2026 16:22
@alespour alespour requested a review from jansimonb March 31, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants