feat(sdp): Updating the ip to endpoint table for sdp by vipul-21 · Pull Request #41548 · cilium/cilium

vipul-21 · 2025-09-05T16:20:26Z

Adds the ip to endpoint info i.e identity and endpoint id in the table.
Adds a watcher that listens for changes in the table and updates the prefixs for faster lookup.
Lookup handlers for DNS proxy for serving the DNS req.
Send the identity<>cidr map from the cilium agent. These cidr are not associated with
an endpoint.

Fixes: #30984
CFP: cilium/design-cfps#54
Relevant PRs: #40982, #41507

feat(sdp): Updating the ip to endpoint table for sdp

vipul-21 · 2025-10-29T22:05:39Z

/test

vipul-21 · 2025-10-31T17:10:15Z

@squeed / @joamaki Friendly ping for reviewing the PR.

squeed

Two small things, then this is good to go.

- Adds the ip to endpoint info i.e identity and endpoint id in the table. - Adds a watcher that listens for changes in the table and updates the prefixs for faster lookup. - Lookup handlers for DNS proxy for serving the DNS req. - Send the identity<>cidr map from the cilium agent. These cidr are not associated with an endpoint. Signed-off-by: Vipul Singh <singhvipul@microsoft.com>

… SDP - Introduce separate tables for prefix-to-identity mapping and IP-to-endpoint mapping. - Update the SDP lookup function to use these new tables when resolving endpoints and identities during DNS request processing. - Modify the existing protobuf definition for the standalone DNS proxy to send the prefix-to-identity mapping separately. Signed-off-by: Vipul Singh <singhvipul@microsoft.com>

Signed-off-by: Vipul Singh <singhvipul@microsoft.com>

vipul-21 · 2025-11-04T23:27:49Z

/test

vipul-21 · 2025-11-05T00:34:07Z

Conformance Gateway API (ci-gateway-api) - open GH issue: #40243 and #42456

vipul-21 · 2025-11-05T00:34:23Z

/ci-gateway-api

vipul-21 · 2025-11-05T00:38:01Z

Cilium E2E Upgrade (ci-e2e-upgrade) - failing due to a warning log that has a retry mechanism:

time=2025-11-04T23:38:20.750321606Z level=warn source=/go/src/github.com/cilium/cilium/pkg/datapath/orchestrator/orchestrator.go:229 msg="Failed to initialize datapath, retrying later" module=agent.datapath.orchestrator error="inserting ipv4 proxy route {proxy 2004 0.0.0.0/0 0 100 false invalid IP 0.0.0.0 0xc0026b80a0 0 universe local {{0 0 } 0 {0}}}: timeout waiting for parameter proxy:2004:0.0.0.0/0 reconciliation: %!w()" retryDelay=10s (1 occurrences)

vipul-21 · 2025-11-05T00:38:15Z

/ci-e2e-upgrade

maintainer-s-little-helper Bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 5, 2025

vipul-21 force-pushed the singhvipul/sdp-6 branch 3 times, most recently from af2e445 to bdc93ac Compare September 9, 2025 23:56

vipul-21 marked this pull request as ready for review September 10, 2025 00:07

vipul-21 requested a review from a team as a code owner September 10, 2025 00:07

vipul-21 requested a review from squeed September 10, 2025 00:07

This comment was marked as outdated.

Sign in to view

vipul-21 marked this pull request as draft September 10, 2025 18:28

vipul-21 force-pushed the singhvipul/sdp-6 branch from bdc93ac to 9061696 Compare September 10, 2025 18:43

vipul-21 marked this pull request as ready for review September 10, 2025 18:44

vipul-21 force-pushed the singhvipul/sdp-6 branch from 9061696 to b602b45 Compare September 10, 2025 19:19

This comment was marked as outdated.

Sign in to view

squeed reviewed Sep 30, 2025

View reviewed changes

Comment thread standalone-dns-proxy/pkg/client/client.go

squeed reviewed Sep 30, 2025

View reviewed changes

Comment thread pkg/fqdn/service/service.go Outdated

squeed requested changes Sep 30, 2025

View reviewed changes

Comment thread standalone-dns-proxy/pkg/client/client.go

Comment thread standalone-dns-proxy/pkg/client/client.go

vipul-21 requested a review from squeed September 30, 2025 21:56

vipul-21 force-pushed the singhvipul/sdp-6 branch from b602b45 to f317722 Compare October 3, 2025 21:45

vipul-21 requested a review from a team as a code owner October 3, 2025 21:45

vipul-21 force-pushed the singhvipul/sdp-6 branch 2 times, most recently from c47c5ff to 09b8fea Compare October 3, 2025 21:50

This comment was marked as outdated.

Sign in to view

vipul-21 force-pushed the singhvipul/sdp-6 branch from 09b8fea to 5dae66c Compare October 6, 2025 18:33

vipul-21 force-pushed the singhvipul/sdp-6 branch from 0d636b6 to 5e68716 Compare October 17, 2025 19:39

This comment was marked as outdated.

Sign in to view

joamaki reviewed Oct 27, 2025

View reviewed changes

Comment thread standalone-dns-proxy/pkg/lookup/lookup.go Outdated

vipul-21 force-pushed the singhvipul/sdp-6 branch from 5e68716 to fea36fc Compare October 29, 2025 21:19

vipul-21 requested a review from joamaki October 29, 2025 21:19

pchaigno added the release-note/misc This PR makes changes that have no direct user impact. label Nov 1, 2025

maintainer-s-little-helper Bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 1, 2025

joamaki approved these changes Nov 4, 2025

View reviewed changes

squeed reviewed Nov 4, 2025

View reviewed changes

Comment thread standalone-dns-proxy/pkg/lookup/lookup_test.go Outdated

maintainer-s-little-helper Bot requested a review from squeed November 4, 2025 20:37

squeed approved these changes Nov 4, 2025

View reviewed changes

maintainer-s-little-helper Bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 4, 2025

vipul-21 added 3 commits November 4, 2025 22:44

send all prefix to id mappings to sdp

868d5ba

Signed-off-by: Vipul Singh <singhvipul@microsoft.com>

vipul-21 force-pushed the singhvipul/sdp-6 branch from fea36fc to 868d5ba Compare November 4, 2025 23:08

squeed added this pull request to the merge queue Nov 5, 2025

Merged via the queue into cilium:main with commit 8904108 Nov 5, 2025
73 checks passed

vipul-21 mentioned this pull request Nov 7, 2025

feat(sdp): Dockerfile for standalone dns proxy #42647

Merged

vipul-21 mentioned this pull request Dec 11, 2025

Adding vipul-21 as a fqdn reviewer cilium/community#361

Merged

5 tasks

cilium-release-bot Bot moved this to Released in cilium v1.19.0 Feb 3, 2026

cilium-release-bot Bot added this to cilium v1.19.0 Feb 3, 2026

Conversation

vipul-21 commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as outdated.

This comment was marked as outdated.

Uh oh!

vipul-21 commented Oct 29, 2025

Uh oh!

vipul-21 commented Oct 31, 2025

Uh oh!

Uh oh!

squeed left a comment

Choose a reason for hiding this comment

Uh oh!

vipul-21 commented Nov 4, 2025

Uh oh!

vipul-21 commented Nov 5, 2025

Uh oh!

vipul-21 commented Nov 5, 2025

Uh oh!

vipul-21 commented Nov 5, 2025

Uh oh!

vipul-21 commented Nov 5, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

vipul-21 commented Sep 5, 2025 •

edited

Loading