[ARVADOS] updated: 2.1.0-659-g1f1c71924

[Git user]

Summary of changes:

  discards  c41f3b2cb785a2d648e8f7746891eb63e345323b (commit)
  discards  123e39f45192118a1e84629bf94a3a7db2bd5014 (commit)
  discards  ba87b0cb04e4a7e8e837567d9a6e317f3d5fa0b2 (commit)
  discards  e758989ebd80f3365baa56be2079d72e286c9cf5 (commit)
  discards  d6c8a79b073cb000c7914e3ef07bf4d95350ae4f (commit)
       via  1f1c719242487831557805b3e73b50eef7a29753 (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (c41f3b2cb785a2d648e8f7746891eb63e345323b)
            \
             N -- N -- N (1f1c719242487831557805b3e73b50eef7a29753)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 1f1c719242487831557805b3e73b50eef7a29753
Author: Nico Cesar <nico at nicocesar.com>
Date:   Thu Apr 15 12:26:42 2021 -0400

    17530: Added check if API host is missing
    
    CallRaw() will have a retry logic so 3s * 20 = 60s to fail,
    and it will be explicit why this is happening.
    
    KeepClient.discoverServices() to make it fail faster
    
    Arvados-DCO-1.1-Signed-off-by: Nico Cesar <nico at curii.com>

diff --git a/sdk/go/arvadosclient/arvadosclient.go b/sdk/go/arvadosclient/arvadosclient.go
index d90c618f7..4c594625e 100644
--- a/sdk/go/arvadosclient/arvadosclient.go
+++ b/sdk/go/arvadosclient/arvadosclient.go
@@ -206,6 +206,9 @@ func (c *ArvadosClient) CallRaw(method string, resourceType string, uuid string,
 	if scheme == "" {
 		scheme = "https"
 	}
+	if c.ApiServer == "" {
+		return nil, fmt.Errorf("Arvados client is not configured (target API host is not set). Maybe env var ARVADOS_API_HOST should be set first?")
+	}
 	u := url.URL{
 		Scheme: scheme,
 		Host:   c.ApiServer}
diff --git a/sdk/go/keepclient/discover.go b/sdk/go/keepclient/discover.go
index 726c3fb30..5eafbbe33 100644
--- a/sdk/go/keepclient/discover.go
+++ b/sdk/go/keepclient/discover.go
@@ -138,6 +138,10 @@ func (kc *KeepClient) discoverServices() error {
 		return nil
 	}
 
+	if kc.Arvados.ApiServer == "" {
+		return fmt.Errorf("Arvados client is not configured (target API host is not set). Maybe env var ARVADOS_API_HOST should be set first?")
+	}
+
 	svcListCacheMtx.Lock()
 	cacheEnt, ok := svcListCache[kc.Arvados.ApiServer]
 	if !ok {

-----------------------------------------------------------------------


hooks/post-receive
--