14199: Fix deadlock in test suite.
authorTom Clegg <tclegg@veritasgenetics.com>
Thu, 4 Oct 2018 18:01:00 +0000 (14:01 -0400)
committerTom Clegg <tclegg@veritasgenetics.com>
Thu, 4 Oct 2018 18:01:00 +0000 (14:01 -0400)
commit005951a5e62a55894eace6e8fb3fe91d5c4ba84c
treee610f0a5608badff59f60f939f376d4228b5865b
parentfde27ce0e46521db9828c228e7fb531e003724a8
14199: Fix deadlock in test suite.

If a KeepClient has been created using an API server address that is
no longer reachable, calling keepclient.RefreshServiceDiscovery() puts
the poll() goroutine into an endless retry loop, and a second call
never returns because ent.clear is never ready to receive.

Work around this in the pull worker tests by only refreshing services
from the API server actually being used, not on additional ones
referenced by previous test cases.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veritasgenetics.com>
sdk/go/keepclient/discover.go
services/keepstore/pull_worker_integration_test.go