Skip to content

fix(helm): remove broken hubble-ui backend health probes - #2601

Open
Sébastien Larivière (slariviere) wants to merge 2 commits into
microsoft:mainfrom
slariviere:fix/hubble-ui-backend-startup-probe
Open

fix(helm): remove broken hubble-ui backend health probes#2601
Sébastien Larivière (slariviere) wants to merge 2 commits into
microsoft:mainfrom
slariviere:fix/hubble-ui-backend-startup-probe

Conversation

@slariviere

@slariviere Sébastien Larivière (slariviere) commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Description

What

Removes the hubble.ui.backend.livenessProbe / readinessProbe values
and the associated startupProbe/livenessProbe/readinessProbe templates
for the hubble-ui backend container.

Why

The backend container is a gRPC-only server on port 8090 with no HTTP
endpoints. The probes were configured as httpGet /healthz, which never
works — enabling them causes the container to CrashLoopBackOff.

Upstream cilium hit the same issue (cilium/cilium#43501) and fixed it by
removing the feature outright in cilium/cilium@b6e50b3, since there's no
working HTTP health endpoint to probe. This PR aligns with that upstream
fix.

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

@slariviere Sébastien Larivière (slariviere) changed the title fix(helm): add startupProbe for hubble-ui backend to tolerate slow hubble-relay startup fix(helm): add startupProbe for hubble-ui backend Jul 29, 2026
@github-actions

Copy link
Copy Markdown

This PR will be closed in 7 days due to inactivity.

@github-actions github-actions Bot added the meta/waiting-for-author Blocked and waiting on the author label Aug 29, 2026
@slariviere

Copy link
Copy Markdown
Contributor Author

Timothy J. Raymond (@timraymond) Qingchuan Hao (@mainred), is there anything we can do to move this PR forward?

@github-actions github-actions Bot removed the meta/waiting-for-author Blocked and waiting on the author label Sep 1, 2026
@nddq

Copy link
Copy Markdown
Member

upstream removed this a while back cilium/cilium@b6e50b3

@slariviere Sébastien Larivière (slariviere) changed the title fix(helm): add startupProbe for hubble-ui backend fix(helm): remove broken hubble-ui backend health probes Sep 1, 2026
@slariviere
Sébastien Larivière (slariviere) deleted the fix/hubble-ui-backend-startup-probe branch September 1, 2026 16:05
@slariviere
Sébastien Larivière (slariviere) restored the fix/hubble-ui-backend-startup-probe branch September 1, 2026 16:07
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@slariviere
Sébastien Larivière (slariviere) deleted the fix/hubble-ui-backend-startup-probe branch September 1, 2026 16:09
@slariviere
Sébastien Larivière (slariviere) restored the fix/hubble-ui-backend-startup-probe branch September 1, 2026 16:10
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@slariviere

Copy link
Copy Markdown
Contributor Author

upstream removed this a while back cilium/cilium@b6e50b3

Quang Nguyen (@nddq) probes removed as upstream

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.

2 participants