Skip to content

Conversation

adisuissa
Copy link
Contributor

Commit Message: xds: add singleton subscription xDS-TP-based config-sources support
Additional Description:
Followup to #39781, as part of #24773.
This PR adds the framework for singleton subscriptions to resources that are defined using the xDS-TP config-sources in the bootstrap.
Next is to convert the current xDS singletons (EDS, RDS, etc.) to use this functionality.

Risk Level: low - not used
Testing: Added unit tests.
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Part of #24773.

Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #39943 was opened by adisuissa.

see: more, trace.

Signed-off-by: Adi Suissa-Peleg <[email protected]>
Signed-off-by: Adi Suissa-Peleg <[email protected]>
@adisuissa adisuissa marked this pull request as ready for review June 18, 2025 20:44
@adisuissa
Copy link
Contributor Author

/assign @nezdolik @tonya11en @yanavlasov

Signed-off-by: Adi Suissa-Peleg <[email protected]>
@adisuissa
Copy link
Contributor Author

Thanks for the review @nezdolik !

@yanavlasov
Copy link
Contributor

LGTM. Waiting for LGTM from @nezdolik as well.

/wait-any

@adisuissa
Copy link
Contributor Author

/retest

@yanavlasov yanavlasov merged commit 96127b2 into envoyproxy:main Jun 27, 2025
25 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.

4 participants