Skip to content

xds: ClusterResolverLoadBalancer handle update for both resolved addresses and errors via ResolutionResult #11997

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
kannanjgithub merged 5 commits into from
Apr 4, 2025
Merged

xds: ClusterResolverLoadBalancer handle update for both resolved addresses and errors via ResolutionResult #11997

kannanjgithub merged 5 commits into from
Apr 4, 2025
+138 −79

Conversation

@kannanjgithub
Copy link
Contributor

@kannanjgithub kannanjgithub commented Apr 4, 2025 •
edited
Loading

Fixes #11995.

@kannanjgithub kannanjgithub requested a review from ejona86 April 4, 2025 11:55
@kannanjgithub kannanjgithub added the TODO:backport PR needs to be backported. Removed after backport complete label Apr 4, 2025
ejona86
ejona86 approved these changes Apr 4, 2025
View reviewed changes
@kannanjgithub kannanjgithub merged commit a13fca2 into grpc:master Apr 4, 2025
16 checks passed
@kannanjgithub kannanjgithub deleted the onResult2Regression branch April 4, 2025 16:38
kannanjgithub added a commit to kannanjgithub/grpc-java that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
7860bad 
…esses and errors via ResolutionResult (grpc#11997)
@kannanjgithub kannanjgithub mentioned this pull request Apr 7, 2025
Merged
kannanjgithub added a commit to kannanjgithub/grpc-java that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
098cd0d 
…esses and errors via ResolutionResult (grpc#11997)
@kannanjgithub kannanjgithub mentioned this pull request Apr 7, 2025
Merged
kannanjgithub added a commit to kannanjgithub/grpc-java that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
87ae513 
…esses and errors via ResolutionResult (grpc#11997)
@kannanjgithub kannanjgithub mentioned this pull request Apr 7, 2025
Merged
kannanjgithub added a commit that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
8681786 
…esses and errors via ResolutionResult (#11997) (#12006)

Backport of #11997 to v1.72.x.
------
Fixes #11995
kannanjgithub added a commit that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
e47748a 
…esses and errors via ResolutionResult (v1.71.x backport) (#12005)

Backport of #11997 to v1.71.x.
------------
Fixes #11995.
kannanjgithub added a commit that referenced this pull request Apr 7, 2025
@kannanjgithub
xds: ClusterResolverLoadBalancer handle update for both resolved addr…
e763891 
…esses and errors via ResolutionResult (v1.70.x backport) (#12004)

Backport of #11997 to v1.70.x.
---------------
Fixes #11995.
@AgraVator AgraVator removed the TODO:backport PR needs to be backported. Removed after backport complete label Apr 15, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@ejona86 ejona86 ejona86 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ClusterResolverLB uses ResolutionResult.getAddresses()

3 participants

@kannanjgithub @ejona86 @AgraVator