Skip to content

Conversation

@markdroth
Copy link
Member

No description provided.

Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note for posterity: it is safe to combine DNS results into a single endpoint because their attributes will be the same. Even with SRV records, since A26, DNS doesn't return the grpclb addresses mixed with the regular addresses.

@markdroth
Copy link
Member Author

@murgatroid99 FYI.

@markdroth markdroth merged commit b1dd022 into grpc:master Mar 6, 2025
1 check passed
@markdroth markdroth deleted the A61_update_logical_dns_in_single_endpoint branch March 6, 2025 22:07
copybara-service bot pushed a commit to grpc/grpc that referenced this pull request Mar 11, 2025
…-coding pick_first (#38248)

See grpc/proposal#477 for context.

Closes #38248

COPYBARA_INTEGRATE_REVIEW=#38248 from markdroth:xds_logical_dns_all_addresses_in_one_endpoint 4cfa6ee
PiperOrigin-RevId: 735820369
ejona86 added a commit to ejona86/grpc-java that referenced this pull request Sep 15, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
ejona86 added a commit to ejona86/grpc-java that referenced this pull request Sep 17, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
ejona86 added a commit to ejona86/grpc-java that referenced this pull request Sep 18, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
ejona86 added a commit to grpc/grpc-java that referenced this pull request Sep 18, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
AgraVator pushed a commit to AgraVator/grpc-java that referenced this pull request Sep 26, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
AgraVator pushed a commit to AgraVator/grpc-java that referenced this pull request Sep 26, 2025
No longer need to hard-code pick_first because of gRFC A61.
grpc/proposal#477
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.

3 participants