[arvados] updated: 2.5.0-314-g026583e07

git repository hosting git at public.arvados.org
Wed Apr 5 19:00:01 UTC 2023


Summary of changes:
 cmd/arvados-client/container_gateway.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

       via  026583e07d9d51ef47fc0c78ba37f78766e2ea3d (commit)
      from  ffc46b6c2fecf585f80c5e4513a088ad21a2185b (commit)

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 026583e07d9d51ef47fc0c78ba37f78766e2ea3d
Author: Tom Clegg <tom at curii.com>
Date:   Wed Apr 5 14:59:26 2023 -0400

    18790: Error out instead of retrying if server is too old.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/cmd/arvados-client/container_gateway.go b/cmd/arvados-client/container_gateway.go
index 2f45e7518..7b39f8e84 100644
--- a/cmd/arvados-client/container_gateway.go
+++ b/cmd/arvados-client/container_gateway.go
@@ -208,7 +208,11 @@ func (lc *logsCommand) copyRange(ctx context.Context, uuid, fnm, byterange strin
 	}
 	defer resp.Body.Close()
 	if resp.StatusCode == http.StatusNotFound {
-		return 0, 0, nil
+		if resp.Header.Get("DAV") == "" {
+			return 0, 0, fmt.Errorf("server does not support container logs API: %s", resp.Status)
+		} else {
+			return 0, 0, nil
+		}
 	}
 	if resp.StatusCode != http.StatusOK && resp.StatusCode != http.StatusPartialContent {
 		body, _ := io.ReadAll(io.LimitReader(resp.Body, 10000))

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list