fix(hubble): bump default hubble-relay image#2229
fix(hubble): bump default hubble-relay image#2229holly-agyei wants to merge 2 commits intomicrosoft:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the default hubble-relay container image used by the retina-hubble Helm chart to a newer upstream version to restore ALPN/h2 support needed by newer grpc-go (and thus Hubble CLI).
Changes:
- Bump default
hubble-relayimage tag fromv1.15.0to1.16.1. - Update the configured digest to match the
1.16.1MCR manifest digest.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| image: | ||
| override: ~ | ||
| repository: "mcr.microsoft.com/oss/cilium/hubble-relay" | ||
| tag: "v1.15.0" | ||
| digest: "sha256:19cd56e7618832257bf88b2f281287cb57f9f7fcb9e04775a6198d4bc4daffae" | ||
| tag: "1.16.1" | ||
| digest: "sha256:641b13bbe9038d7b0bedd6480e3d704174432e1fd05a25728a4eb27ec47df89a" | ||
| useDigest: false | ||
| pullPolicy: "Always" |
There was a problem hiding this comment.
The hubble-relay image digest, useDigest, and override fields in values are currently ineffective: the hubble-relay Deployment template renders the image as {{ .Values.hubble.relay.image.repository }}:{{ .Values.hubble.relay.image.tag }} and never uses these keys. As a result, updating the digest here won’t actually pin the hubble-relay image. Consider updating the hubble-relay template to use the existing cilium.image helper (or otherwise conditionally append @<digest> when useDigest is true and honor override).
Summary
retina-hubblehubble-relayimage fromv1.15.0to1.16.11.16.1MCR manifestcilium.imagehelper sooverride,useDigest, anddigestare honoredFixes #2165.
Verification
1.16.1foross/cilium/hubble-relay1.16.1manifest digest issha256:641b13bbe9038d7b0bedd6480e3d704174432e1fd05a25728a4eb27ec47df89acilium.imagehelper