-
Notifications
You must be signed in to change notification settings - Fork 9.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
integration/clientv3/naming TestEtcdGrpcResolverRoundRobin #15794
Comments
Thanks for raising this @fuweid. This is the test I introduced recently so I will pick up and fix, it looks like the threshold we set needs tuning as roundrobin is never exactly 50/50, from memory currently we allow up to 10% variation so can increase this slightly. |
https://github.com/etcd-io/etcd/actions/runs/4933543352/jobs/8817599646?pr=15865 |
Thanks for highlighting that flake @chaochn47, re-opened. It looks like my last update helped, but still a little more tuning is needed. I will relax the threshold just a little more to Will raise a pr tonight. |
Which github workflows are flaking?
test (linux-amd64-integration-2-cpu)
Which tests are flaking?
integration/clientv3/naming TestEtcdGrpcResolverRoundRobin
Github Action link
https://github.com/etcd-io/etcd/actions/runs/4831119673/jobs/8608139659?pr=15792#logs
#15792 (comment)
Reason for failure (if possible)
No response
Anything else we need to know?
Log
The text was updated successfully, but these errors were encountered: