Skip to content

ClusterResolverLB uses ResolutionResult.getAddresses() #11995

@ejona86

Description

@ejona86

#11330 introduced a StatusOr for addresses within ResolutionResult. It marked the old getAddresses() as deprecated, because it can throw. ManagedChannelImpl was updated in the same PR but xds/build.gradle has -Xlint:-deprecation so the usage in ClusterResolverLB was missed.

This is fairly serious as a DNS failure when LOGICAL_DNS is in use probably would cause a channel panic. We'll want to release patch releases for older versions (at least 1.71 and 1.70).

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions