status-im
/
consul
mirror of https://github.com/status-im/consul.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Drop testutil wait to 2 seconds. 

There's likely a race (related to https://github.com/hashicorp/consul/issues/2644) where the catalog update might be in but the leader tracking doesn't report a leader, so this blocks forever and then times out. As a workaround we can lower the query wait time to always allow for a few retries.
This commit is contained in:
James Phillips 2017-02-06 11:52:00 -08:00 committed by GitHub
parent 97dbfb8b32
commit 14c6d009cc
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
testutil/server.go
View File

@ -298,7 +298,7 @@ func (s *TestServer) waitForLeader() {
var index int64 var index int64
WaitForResult(func() (bool, error) { WaitForResult(func() (bool, error) {
// Query the API and check the status code. // Query the API and check the status code.
url := s.url(fmt.Sprintf("/v1/catalog/nodes?index=%d&wait=10s", index)) url := s.url(fmt.Sprintf("/v1/catalog/nodes?index=%d&wait=2s", index))
resp, err := s.HttpClient.Get(url) resp, err := s.HttpClient.Get(url)
if err != nil { if err != nil {
return false, err return false, err