Skip to content

fix(chart): route hubble ui api via grpc_pass - #2453

Open
abdullazin wants to merge 4 commits into
microsoft:mainfrom
slariviere:INFRA-8293-hubble-ui-backend-route
Open

fix(chart): route hubble ui api via grpc_pass#2453
abdullazin wants to merge 4 commits into
microsoft:mainfrom
slariviere:INFRA-8293-hubble-ui-backend-route

Conversation

@abdullazin

@abdullazin abdullazin commented Jun 18, 2026

Copy link
Copy Markdown

Summary

  • fix the Hubble UI /api path in nginx by switching from proxy_pass to grpc_pass
  • preserve existing baseUrl rewrite logic
  • keep the fix scoped to nginx template behavior (no extra backend service/routing objects)

Why this change

  • with Envoy in front of Hubble UI, the UI data stream call to /api/ui.UI/GetEvents was failing with HTTP 400
  • the UI surfaced this as: Data stream has failed on the UI backend: Http response at 400 or 500 level, http status code: 400
  • the backend response included: grpc-message: gRPC requires HTTP/2
  • this indicates the /api path needed to be handled as gRPC traffic at the nginx hop

Path handling note

  • a gRPC request carries its route in the :path pseudo-header
  • grpc_pass does not rewrite request path based on upstream URI suffix
  • therefore the conditional grpc://127.0.0.1:8090 vs grpc://127.0.0.1:8090/ is unnecessary
  • this PR simplifies to a single grpc_pass grpc://127.0.0.1:8090;

Files

  • deploy/hubble/manifests/controller/helm/retina/templates/hubble-ui/_nginx.tpl

Verification

  • rendered chart templates locally after the nginx change
  • confirmed only _nginx.tpl changed in this PR branch

@abdullazin
abdullazin force-pushed the INFRA-8293-hubble-ui-backend-route branch from cd25063 to a4e2812 Compare June 18, 2026 14:47
@abdullazin

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree [company="GoTo"]

@abdullazin

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="GoTo"

@abdullazin
abdullazin marked this pull request as ready for review June 18, 2026 14:51
@abdullazin
abdullazin requested a review from a team as a code owner June 18, 2026 14:51
@abdullazin
abdullazin marked this pull request as draft June 18, 2026 15:38
@abdullazin
abdullazin marked this pull request as ready for review June 18, 2026 15:55
@abdullazin
abdullazin marked this pull request as draft June 22, 2026 20:01
@abdullazin
abdullazin force-pushed the INFRA-8293-hubble-ui-backend-route branch from c5752f7 to de5889e Compare June 22, 2026 20:50
@abdullazin abdullazin changed the title feat(chart): route hubble ui api directly to backend service fix(chart): route hubble ui api via grpc_pass Jun 22, 2026
@abdullazin
abdullazin marked this pull request as ready for review June 22, 2026 20:54
@abdullazin

Copy link
Copy Markdown
Author

@nairashu or @alam-tahmid , can you please review my PR?

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.

1 participant