[arvados] updated: 2.6.0-74-gdba8db406

git repository hosting git at public.arvados.org
Fri Apr 21 13:26:42 UTC 2023


Summary of changes:
 cmd/arvados-client/container_gateway.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

       via  dba8db406b1b63173ae05d0baa3f5f8f1a48db17 (commit)
      from  caa54be9b2984ce2d08dcc6ad377e7f0be339dae (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 dba8db406b1b63173ae05d0baa3f5f8f1a48db17
Author: Tom Clegg <tom at curii.com>
Date:   Fri Apr 21 09:26:02 2023 -0400

    18790: Fix error handling.
    
    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 e4d1141a0..08f37d564 100644
--- a/cmd/arvados-client/container_gateway.go
+++ b/cmd/arvados-client/container_gateway.go
@@ -170,12 +170,9 @@ poll:
 		}
 		if checkState {
 			cr, err = rpcconn.ContainerRequestGet(ctx, arvados.GetOptions{UUID: crUUID, Select: []string{"uuid", "container_uuid", "state"}})
-			if err != nil {
-				return fmt.Errorf("error retrieving %s: %w", crUUID, err)
-			}
 			if err != nil {
 				if !anySuccess {
-					return err
+					return fmt.Errorf("error retrieving %s: %w", crUUID, err)
 				}
 				fmt.Fprintln(stderr, err)
 				delay = pollInterval

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list