Skip to content

Commit 13cc209

Browse files
ahrtrk8s-infra-cherrypick-robot
authored andcommitted
Replace resolver.State.Addresses with resolver.State.Endpoint.Addresses
Signed-off-by: Benjamin Wang <[email protected]>
1 parent 005880d commit 13cc209

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

client/v3/internal/resolver/resolver.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,15 @@ func (r *EtcdManualResolver) SetEndpoints(endpoints []string) {
6060

6161
func (r EtcdManualResolver) updateState() {
6262
if r.CC != nil {
63-
addresses := make([]resolver.Address, len(r.endpoints))
63+
eps := make([]resolver.Endpoint, len(r.endpoints))
6464
for i, ep := range r.endpoints {
6565
addr, serverName := endpoint.Interpret(ep)
66-
addresses[i] = resolver.Address{Addr: addr, ServerName: serverName}
66+
eps[i] = resolver.Endpoint{Addresses: []resolver.Address{
67+
{Addr: addr, ServerName: serverName},
68+
}}
6769
}
6870
state := resolver.State{
69-
Addresses: addresses,
71+
Endpoints: eps,
7072
ServiceConfig: r.serviceConfig,
7173
}
7274
r.UpdateState(state)

client/v3/naming/resolver/resolver.go

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -86,22 +86,26 @@ func (r *resolver) watch() {
8686
}
8787
}
8888

89-
addrs := convertToGRPCAddress(allUps)
90-
r.cc.UpdateState(gresolver.State{Addresses: addrs})
89+
eps := convertToGRPCEndpoint(allUps)
90+
r.cc.UpdateState(gresolver.State{Endpoints: eps})
9191
}
9292
}
9393
}
9494

95-
func convertToGRPCAddress(ups map[string]*endpoints.Update) []gresolver.Address {
96-
var addrs []gresolver.Address
95+
func convertToGRPCEndpoint(ups map[string]*endpoints.Update) []gresolver.Endpoint {
96+
var eps []gresolver.Endpoint
9797
for _, up := range ups {
98-
addr := gresolver.Address{
99-
Addr: up.Endpoint.Addr,
100-
Metadata: up.Endpoint.Metadata,
98+
ep := gresolver.Endpoint{
99+
Addresses: []gresolver.Address{
100+
{
101+
Addr: up.Endpoint.Addr,
102+
Metadata: up.Endpoint.Metadata,
103+
},
104+
},
101105
}
102-
addrs = append(addrs, addr)
106+
eps = append(eps, ep)
103107
}
104-
return addrs
108+
return eps
105109
}
106110

107111
// ResolveNow is a no-op here.

0 commit comments

Comments
 (0)