Skip to content

fix(helm): use native gRPC probe on relay :4222 and bump default to v1.19.5#2500

Merged
nddq merged 1 commit into
microsoft:mainfrom
slariviere:fix/hubble-relay-native-grpc-probe-comments
Jul 23, 2026
Merged

fix(helm): use native gRPC probe on relay :4222 and bump default to v1.19.5#2500
nddq merged 1 commit into
microsoft:mainfrom
slariviere:fix/hubble-relay-native-grpc-probe-comments

Conversation

@slariviere

@slariviere slariviere commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Closes #2165

Address review comments from #2282 (closed due to inactivity)

Summary

Fixes hubble-relay's startup/readiness/liveness probes, which fail on Cilium
v1.16+ images because the grpc_health_probe binary used by the legacy
exec-based probe was removed upstream (cilium/cilium#37806).

  1. templates/hubble-relay/deployment.yaml — single native grpc: probe
    against the relay's dedicated health server (hubble.relay.grpcHealthPort,
    default 4222), replacing the k8s-version-gated exec/native branching.
    The health listener is always plaintext regardless of hubble.tls.enabled
    (only the main API on listenPort is TLS-wrapped), so this probe works in
    both TLS and non-TLS deployments. Also renders the relay image via the
    cilium.image helper (previously a raw repository:tag string) so
    useDigest/override are actually honored.
  2. templates/hubble-relay/configmap.yaml — wires hubble.relay.grpcHealthPort
    through to health-listen-address, so the relay's health server actually
    listens on the port the probes target.
  3. templates/hubble-ui/deployment.yaml — frontend and backend images now
    also render via the cilium.image helper.
  4. values.yaml:
    • hubble.relay.image default bumped from
      mcr.microsoft.com/oss/cilium/hubble-relay:v1.15.0 to
      quay.io/cilium/hubble-relay:v1.19.5 (matches the Cilium v1.19.3
      dependency in go.mod), with useDigest: true.
    • Added hubble.relay.grpcHealthPort: "4222".
    • hubble.ui.backend.image and hubble.ui.frontend.image both moved from
      mcr.microsoft.com to quay.io/cilium/hubble-ui{,-backend}:v0.13.5
      (previously split across two registries at v0.12.2), matching upstream
      Cilium's v1.19.5 chart image pins.

@slariviere
slariviere requested a review from a team as a code owner July 6, 2026 15:53
Comment thread deploy/hubble/manifests/controller/helm/retina/values.yaml
Comment thread deploy/hubble/manifests/controller/helm/retina/values.yaml
Comment thread deploy/hubble/manifests/controller/helm/retina/values.yaml Outdated
Comment thread deploy/hubble/manifests/controller/helm/retina/values.yaml Outdated
Comment thread deploy/hubble/manifests/controller/helm/retina/values.yaml
@slariviere
slariviere force-pushed the fix/hubble-relay-native-grpc-probe-comments branch from cacacfa to 1c9ffff Compare July 23, 2026 16:04
@slariviere
slariviere requested a review from nddq July 23, 2026 16:06
@nddq nddq changed the title fix(helm): use native gRPC probe on relay :4222 and bump default to v1.17.15 fix(helm): use native gRPC probe on relay :4222 and bump default to v1.19.5 Jul 23, 2026
@nddq
nddq enabled auto-merge July 23, 2026 20:53

@nddq nddq left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm, thanks!

@nddq

nddq commented Jul 23, 2026

Copy link
Copy Markdown
Member

@slariviere All commits need to have verified signatures

auto-merge was automatically disabled July 23, 2026 21:05

Head branch was pushed to by a user without write access

@slariviere
slariviere force-pushed the fix/hubble-relay-native-grpc-probe-comments branch from 1a3cd95 to 0289ff9 Compare July 23, 2026 21:05
@slariviere

Copy link
Copy Markdown
Contributor Author

@nddq should be good now

@nddq
nddq enabled auto-merge July 23, 2026 21:16
@nddq
nddq added this pull request to the merge queue Jul 23, 2026
Merged via the queue into microsoft:main with commit 5b41eae Jul 23, 2026
31 checks passed
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.

hubble-relay image v1.15.0 missing h2 ALPN support that breaks hubble CLI (grpc-go >= 1.67)

2 participants