Open
Conversation
nddq
changed the title
nddq
force-pushed
the
wip/cilium+IGUpgrades
branch
3 times, most recently
from
539261a to
7a62f50
Compare
nddq
changed the title
nddq
force-pushed
the
wip/cilium+IGUpgrades
branch
from
7a62f50 to
5a7c018
Compare
This was referenced Feb 11, 2026
nddq
force-pushed
the
wip/cilium+IGUpgrades
branch
5 times, most recently
from
96e3bf9 to
cdee4b8
Compare
Upgrade the Cilium dependency from v1.18.0-pre.1 to v1.19.0 and adapt the codebase to breaking API changes. Key changes: - Rewrite Hubble flow enrichment and DNS parsing to work with Cilium v1.19 APIs (structpb-based extensions replace RetinaMetadata) - Add DI stubs for new Cilium cell/hive requirements - Resolve duplicate workqueue metrics providers - Add new eBPF-based DNS and TCP retransmission plugins - Bump Go base images to 1.25.7 and go directive to 1.25.0 - Remove cross-compilation from Dockerfiles for native arm64 builds via QEMU (Microsoft Go 1.25 requires CGO_ENABLED=1 for crypto) - Fix prometheus metric parsing with prometheus/common v0.67.5 - Upgrade golangci-lint from v1 to v2.9.0; set CGO_ENABLED=0 for lint to prevent typecheck errors from _cprog C source directories - Fix DNS metrics by moving recordsChannel creation outside EnablePodLevel conditional Refs: microsoft#1788 Signed-off-by: Quang Nguyen <nguyenquang@microsoft.com>
nddq
force-pushed
the
wip/cilium+IGUpgrades
branch
from
cdee4b8 to
93c3d87
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Upgrade the Cilium dependency from
v1.18.0-pre.1tov1.19.0and adapt the codebase to breaking API changes. This includes:structpb-based extensions replaceRetinaMetadata)CGO_ENABLED=1for its crypto backend)prometheus/commonv0.67.5CGO_ENABLED=0for lint to prevent typecheck errors from_cprogC source directoriesrecordsChannelcreation outsideEnablePodLevelconditional to fix DNS metrics in basic modeRelated Issue
Refs #1788
Checklist
git commit -S -s ...). See this documentation on signing commits.Screenshots (if applicable) or Testing Completed
CGO_ENABLED=0Additional Notes
This is a major dependency upgrade. Cilium v1.19.0 introduces breaking changes in the Hubble and cell APIs that required significant adaptation in the Retina hubble control plane. This PR is for CI validation only — it will be superseded by smaller, reviewable PRs once CI is confirmed green.
Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.