X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/55aafbb07904ca24390dd47ea960eae7cb2b909a..397981dadc145225c691c8643b10527c9710f1fb:/sdk/go/keepclient/discover.go diff --git a/sdk/go/keepclient/discover.go b/sdk/go/keepclient/discover.go index 2140dceabb..726c3fb30c 100644 --- a/sdk/go/keepclient/discover.go +++ b/sdk/go/keepclient/discover.go @@ -6,6 +6,7 @@ package keepclient import ( "encoding/json" + "errors" "fmt" "log" "os" @@ -15,20 +16,27 @@ import ( "syscall" "time" - "git.curoverse.com/arvados.git/sdk/go/arvadosclient" + "git.arvados.org/arvados.git/sdk/go/arvadosclient" ) -// ClearCache clears the Keep service discovery cache. +// RefreshServiceDiscovery clears the Keep service discovery cache. func RefreshServiceDiscovery() { + var wg sync.WaitGroup + defer wg.Wait() svcListCacheMtx.Lock() defer svcListCacheMtx.Unlock() for _, ent := range svcListCache { - ent.clear <- struct{}{} + wg.Add(1) + clear := ent.clear + go func() { + clear <- struct{}{} + wg.Done() + }() } } -// ClearCacheOnSIGHUP installs a signal handler that calls -// ClearCache when SIGHUP is received. +// RefreshServiceDiscoveryOnSIGHUP installs a signal handler that calls +// RefreshServiceDiscovery when SIGHUP is received. func RefreshServiceDiscoveryOnSIGHUP() { svcListCacheMtx.Lock() defer svcListCacheMtx.Unlock() @@ -144,7 +152,22 @@ func (kc *KeepClient) discoverServices() error { } svcListCacheMtx.Unlock() - return kc.loadKeepServers(<-cacheEnt.latest) + select { + case <-time.After(time.Minute): + return errors.New("timed out while getting initial list of keep services") + case sl := <-cacheEnt.latest: + return kc.loadKeepServers(sl) + } +} + +func (kc *KeepClient) RefreshServiceDiscovery() { + svcListCacheMtx.Lock() + ent, ok := svcListCache[kc.Arvados.ApiServer] + svcListCacheMtx.Unlock() + if !ok || kc.Arvados.KeepServiceURIs != nil || kc.disableDiscovery { + return + } + ent.clear <- struct{}{} } // LoadKeepServicesFromJSON gets list of available keep services from