[arvados] updated: 2.7.0-5931-g5b0d52ac34

git repository hosting git at public.arvados.org
Wed Jan 31 16:38:59 UTC 2024


Summary of changes:
 sdk/go/arvados/client.go | 5 +++++
 1 file changed, 5 insertions(+)

       via  5b0d52ac3485c84de8df559ac9efaba5c4433135 (commit)
      from  67068b56fcbe322f04d0107dc6c19a6b12e248fa (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 5b0d52ac3485c84de8df559ac9efaba5c4433135
Author: Tom Clegg <tom at curii.com>
Date:   Wed Jan 31 11:37:23 2024 -0500

    Add comment to workaround while waiting for upstream to merge PR.
    
    refs #21252
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/sdk/go/arvados/client.go b/sdk/go/arvados/client.go
index 991de1caa9..7bc3d5bc42 100644
--- a/sdk/go/arvados/client.go
+++ b/sdk/go/arvados/client.go
@@ -307,6 +307,11 @@ func (c *Client) Do(req *http.Request) (*http.Response, error) {
 		if c.Timeout == 0 {
 			return false, nil
 		}
+		// This check can be removed when
+		// https://github.com/hashicorp/go-retryablehttp/pull/210
+		// (or equivalent) is merged and we update go.mod.
+		// Until then, it is needed to pass
+		// TestNonRetryableStdlibError.
 		if respErr != nil && reqErrorRe.MatchString(respErr.Error()) {
 			return false, nil
 		}

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list