X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/75e00445b6de230493e9ee37483dd4c469db29b1..68438af4765de8fad80a4a770d9bafeb615aa6c0:/sdk/go/keepclient/discover.go diff --git a/sdk/go/keepclient/discover.go b/sdk/go/keepclient/discover.go index e2cd329fc4..62936e7183 100644 --- a/sdk/go/keepclient/discover.go +++ b/sdk/go/keepclient/discover.go @@ -1,7 +1,12 @@ +// Copyright (C) The Arvados Authors. All rights reserved. +// +// SPDX-License-Identifier: Apache-2.0 + package keepclient import ( "encoding/json" + "errors" "fmt" "log" "os" @@ -16,10 +21,16 @@ import ( // ClearCache 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) + go func() { + ent.clear <- struct{}{} + wg.Done() + }() } } @@ -140,7 +151,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