Browse Source

*: log, expect by capability check

Gyu-Ho Lee 9 years ago
parent
commit
d78345244b
3 changed files with 2 additions and 5 deletions
  1. 0 4
      e2e/ctl_v2_test.go
  2. 1 1
      e2e/etcd_test.go
  3. 1 0
      etcdserver/api/v2http/capability.go

+ 0 - 4
e2e/ctl_v2_test.go

@@ -169,10 +169,6 @@ func testCtlV2GetRoleUser(t *testing.T, cfg *etcdProcessClusterConfig) {
 		}
 		}
 	}()
 	}()
 
 
-	// wait for the server capabilities to be updated based on the version;
-	// the update loop has a delay of 500ms, so 1s should be enough wait time
-	time.Sleep(time.Second)
-
 	if err := etcdctlRoleAdd(epc, "foo"); err != nil {
 	if err := etcdctlRoleAdd(epc, "foo"); err != nil {
 		t.Fatalf("failed to add role (%v)", err)
 		t.Fatalf("failed to add role (%v)", err)
 	}
 	}

+ 1 - 1
e2e/etcd_test.go

@@ -252,7 +252,7 @@ func newEtcdProcessCluster(cfg *etcdProcessClusterConfig) (*etcdProcessCluster,
 
 
 	// wait for cluster to start
 	// wait for cluster to start
 	readyC := make(chan error, cfg.clusterSize+cfg.proxySize)
 	readyC := make(chan error, cfg.clusterSize+cfg.proxySize)
-	readyStr := "membership: set the initial cluster version to"
+	readyStr := "enabled capabilities for version"
 	for i := range etcdCfgs {
 	for i := range etcdCfgs {
 		go func(etcdp *etcdProcess) {
 		go func(etcdp *etcdProcess) {
 			rs := readyStr
 			rs := readyStr

+ 1 - 0
etcdserver/api/v2http/capability.go

@@ -63,6 +63,7 @@ func capabilityLoop(s *etcdserver.EtcdServer) {
 			enableMapMu.Lock()
 			enableMapMu.Lock()
 			enabledMap = capabilityMaps[pv.String()]
 			enabledMap = capabilityMaps[pv.String()]
 			enableMapMu.Unlock()
 			enableMapMu.Unlock()
+			plog.Infof("enabled capabilities for version %s", pv)
 		}
 		}
 
 
 		select {
 		select {