Skip to content

Commit 8f933a5

Browse files
authored
Merge pull request #19780 from ahrtr/state_addr_20250422
Replace `resolver.State.Addresses` with `resolver.State.Endpoint.Addresses`
2 parents 27ff3ec + 09f897d commit 8f933a5

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

client/v3/internal/resolver/resolver.go

+5-3
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,15 @@ func (r *EtcdManualResolver) SetEndpoints(endpoints []string) {
6161

6262
func (r EtcdManualResolver) updateState() {
6363
if r.CC != nil {
64-
addresses := make([]resolver.Address, len(r.endpoints))
64+
eps := make([]resolver.Endpoint, len(r.endpoints))
6565
for i, ep := range r.endpoints {
6666
addr, serverName := endpoint.Interpret(ep)
67-
addresses[i] = resolver.Address{Addr: addr, ServerName: serverName}
67+
eps[i] = resolver.Endpoint{Addresses: []resolver.Address{
68+
{Addr: addr, ServerName: serverName},
69+
}}
6870
}
6971
state := resolver.State{
70-
Addresses: addresses,
72+
Endpoints: eps,
7173
ServiceConfig: r.serviceConfig,
7274
}
7375
r.UpdateState(state)

client/v3/naming/resolver/resolver.go

+13-9
Original file line numberDiff line numberDiff line change
@@ -100,22 +100,26 @@ func (r *resolver) watch() {
100100
}
101101
}
102102

103-
addrs := convertToGRPCAddress(allUps)
104-
r.cc.UpdateState(gresolver.State{Addresses: addrs})
103+
eps := convertToGRPCEndpoint(allUps)
104+
r.cc.UpdateState(gresolver.State{Endpoints: eps})
105105
}
106106
}
107107
}
108108

109-
func convertToGRPCAddress(ups map[string]*endpoints.Update) []gresolver.Address {
110-
var addrs []gresolver.Address
109+
func convertToGRPCEndpoint(ups map[string]*endpoints.Update) []gresolver.Endpoint {
110+
var eps []gresolver.Endpoint
111111
for _, up := range ups {
112-
addr := gresolver.Address{
113-
Addr: up.Endpoint.Addr,
114-
Metadata: up.Endpoint.Metadata,
112+
ep := gresolver.Endpoint{
113+
Addresses: []gresolver.Address{
114+
{
115+
Addr: up.Endpoint.Addr,
116+
Metadata: up.Endpoint.Metadata,
117+
},
118+
},
115119
}
116-
addrs = append(addrs, addr)
120+
eps = append(eps, ep)
117121
}
118-
return addrs
122+
return eps
119123
}
120124

121125
// ResolveNow is a no-op here.

0 commit comments

Comments
 (0)