mirror of
https://github.com/etcd-io/etcd.git
synced 2024-09-27 06:25:44 +00:00

If grpc starts with `--resolver-prefix`, its memberlist will return all alive proxy nodes. When one GRPC proxy node goes down it shouldn't return that node, however, it's returned. Backport of PR #15907 / commit ecfed91e5050431385961573e5c09658a66a59d5. Co-authored-by: yellowzf <zzhf3311@163.com> Signed-off-by: Ivan Valdes <ivan@vald.es>
172 lines
4.7 KiB
Go
172 lines
4.7 KiB
Go
// Copyright 2017 The etcd Authors
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package grpcproxy
|
|
|
|
import (
|
|
"context"
|
|
"net"
|
|
"os"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"go.etcd.io/etcd/clientv3"
|
|
"go.etcd.io/etcd/clientv3/naming/endpoints"
|
|
pb "go.etcd.io/etcd/etcdserver/etcdserverpb"
|
|
"go.etcd.io/etcd/integration"
|
|
"go.etcd.io/etcd/pkg/testutil"
|
|
|
|
"google.golang.org/grpc"
|
|
)
|
|
|
|
func TestClusterProxyMemberList(t *testing.T) {
|
|
defer testutil.AfterTest(t)
|
|
|
|
clus := integration.NewClusterV3(t, &integration.ClusterConfig{Size: 1})
|
|
defer clus.Terminate(t)
|
|
|
|
serverEps := []string{clus.Members[0].GRPCAddr()}
|
|
prefix := "test-prefix"
|
|
hostname, _ := os.Hostname()
|
|
cts := newClusterProxyServer(serverEps, prefix, t)
|
|
defer cts.close(t)
|
|
|
|
cfg := clientv3.Config{
|
|
Endpoints: []string{cts.caddr},
|
|
DialTimeout: 5 * time.Second,
|
|
}
|
|
client, err := clientv3.New(cfg)
|
|
if err != nil {
|
|
t.Fatalf("err %v, want nil", err)
|
|
}
|
|
defer client.Close()
|
|
|
|
// wait some time for register-loop to write keys
|
|
time.Sleep(200 * time.Millisecond)
|
|
|
|
var mresp *clientv3.MemberListResponse
|
|
mresp, err = client.Cluster.MemberList(context.Background())
|
|
if err != nil {
|
|
t.Fatalf("err %v, want nil", err)
|
|
}
|
|
|
|
if len(mresp.Members) != 1 {
|
|
t.Fatalf("len(mresp.Members) expected 1, got %d (%+v)", len(mresp.Members), mresp.Members)
|
|
}
|
|
if len(mresp.Members[0].ClientURLs) != 1 {
|
|
t.Fatalf("len(mresp.Members[0].ClientURLs) expected 1, got %d (%+v)", len(mresp.Members[0].ClientURLs), mresp.Members[0].ClientURLs[0])
|
|
}
|
|
assert.Contains(t, mresp.Members, &pb.Member{Name: hostname, ClientURLs: []string{cts.caddr}})
|
|
|
|
//test proxy member add
|
|
newMemberAddr := "127.0.0.2:6789"
|
|
Register(cts.c, prefix, newMemberAddr, 7)
|
|
// wait some time for proxy update members
|
|
time.Sleep(200 * time.Millisecond)
|
|
|
|
//check add member succ
|
|
mresp, err = client.Cluster.MemberList(context.Background())
|
|
if err != nil {
|
|
t.Fatalf("err %v, want nil", err)
|
|
}
|
|
if len(mresp.Members) != 2 {
|
|
t.Fatalf("len(mresp.Members) expected 2, got %d (%+v)", len(mresp.Members), mresp.Members)
|
|
}
|
|
assert.Contains(t, mresp.Members, &pb.Member{Name: hostname, ClientURLs: []string{newMemberAddr}})
|
|
|
|
//test proxy member delete
|
|
deregisterMember(cts.c, prefix, newMemberAddr, t)
|
|
// wait some time for proxy update members
|
|
time.Sleep(200 * time.Millisecond)
|
|
|
|
//check delete member succ
|
|
mresp, err = client.Cluster.MemberList(context.Background())
|
|
if err != nil {
|
|
t.Fatalf("err %v, want nil", err)
|
|
}
|
|
if len(mresp.Members) != 1 {
|
|
t.Fatalf("len(mresp.Members) expected 1, got %d (%+v)", len(mresp.Members), mresp.Members)
|
|
}
|
|
assert.Contains(t, mresp.Members, &pb.Member{Name: hostname, ClientURLs: []string{cts.caddr}})
|
|
}
|
|
|
|
type clusterproxyTestServer struct {
|
|
cp pb.ClusterServer
|
|
c *clientv3.Client
|
|
server *grpc.Server
|
|
l net.Listener
|
|
donec <-chan struct{}
|
|
caddr string
|
|
}
|
|
|
|
func (cts *clusterproxyTestServer) close(t *testing.T) {
|
|
cts.server.Stop()
|
|
cts.l.Close()
|
|
cts.c.Close()
|
|
select {
|
|
case <-cts.donec:
|
|
return
|
|
case <-time.After(5 * time.Second):
|
|
t.Fatalf("register-loop took too long to return")
|
|
}
|
|
}
|
|
|
|
func newClusterProxyServer(endpoints []string, prefix string, t *testing.T) *clusterproxyTestServer {
|
|
cfg := clientv3.Config{
|
|
Endpoints: endpoints,
|
|
DialTimeout: 5 * time.Second,
|
|
}
|
|
client, err := clientv3.New(cfg)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
cts := &clusterproxyTestServer{
|
|
c: client,
|
|
}
|
|
cts.l, err = net.Listen("tcp", "127.0.0.1:0")
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
var opts []grpc.ServerOption
|
|
cts.server = grpc.NewServer(opts...)
|
|
servec := make(chan struct{})
|
|
go func() {
|
|
<-servec
|
|
cts.server.Serve(cts.l)
|
|
}()
|
|
|
|
Register(client, prefix, cts.l.Addr().String(), 7)
|
|
cts.cp, cts.donec = NewClusterProxy(client, cts.l.Addr().String(), prefix)
|
|
cts.caddr = cts.l.Addr().String()
|
|
pb.RegisterClusterServer(cts.server, cts.cp)
|
|
close(servec)
|
|
|
|
// wait some time for free port 0 to be resolved
|
|
time.Sleep(500 * time.Millisecond)
|
|
|
|
return cts
|
|
}
|
|
|
|
func deregisterMember(c *clientv3.Client, prefix, addr string, t *testing.T) {
|
|
em, err := endpoints.NewManager(c, prefix)
|
|
if err != nil {
|
|
t.Fatalf("new endpoint manager failed, err %v", err)
|
|
}
|
|
if err = em.DeleteEndpoint(c.Ctx(), prefix+"/"+addr); err != nil {
|
|
t.Fatalf("delete endpoint failed, err %v", err)
|
|
}
|
|
}
|