gojek / weaver

An Advanced HTTP Reverse Proxy with Dynamic Sharding Strategies
Apache License 2.0
602 stars 53 forks source link

Sometimes tests are stuck for longer time. #24

Closed gwthm-in closed 5 years ago

gwthm-in commented 5 years ago

Sometimes when running tests, they are stuck for longer time, couldn't get them to complete.

╭─TheInsider ~/Documents/repos/gojektech/weaver ‹ruby-2.5.0› ‹weaver-cli*›
╰─➤  mk test
cp weaver.conf.yaml.sample weaver.conf.yaml
GO111MODULE=on go test ./...
ok      github.com/gojektech/weaver (cached)
?       github.com/gojektech/weaver/cmd/weaver  [no test files]
ok      github.com/gojektech/weaver/config  0.021s
^C{"level":"info","msg":"bootstrapping router using etcd on /weaver/acls/","time":"2019-03-19T07:53:25+05:30"}
{"level":"info","msg":"creating router namespace on etcd using /weaver/acls/","time":"2019-03-19T07:53:25+05:30"}
FAIL    github.com/gojektech/weaver/etcd    14.251s
make: *** [Makefile:40: test] Error 1

Here is the message, that I got when trying to reproduce it.

MuthukumarHelios commented 5 years ago

I think this is because of viper Config code ref If you fake your viper call the test case will pass

rShetty commented 5 years ago

Closing this, this is issue with connectivity to etcd. please check the etcd connectivity and reraise the issue, if the issue persists