[ARVADOS] updated: 1.1.0-195-g1f927c1

Git user git at public.curoverse.com
Wed Nov 29 12:12:46 EST 2017


Summary of changes:
 services/crunch-run/crunchrun.go | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

       via  1f927c19945fcdf02be92272cab888d512cb0a37 (commit)
      from  fbcedc4041edae0c1ff5367915bb75d60d47a99f (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 1f927c19945fcdf02be92272cab888d512cb0a37
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Wed Nov 29 12:12:22 2017 -0500

    12614: Restore keepclient error check prior to NewContainerRunner()
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/services/crunch-run/crunchrun.go b/services/crunch-run/crunchrun.go
index 180da0c..fc0dda7 100644
--- a/services/crunch-run/crunchrun.go
+++ b/services/crunch-run/crunchrun.go
@@ -1639,6 +1639,11 @@ func main() {
 	api.Retries = 8
 
 	kc, kcerr := keepclient.MakeKeepClient(api)
+	if kcerr != nil {
+		log.Fatalf("%s: %v", containerId, kcerr)
+	}
+	kc.BlockCache = &keepclient.BlockCache{MaxBlocks: 2}
+	kc.Retries = 4
 
 	// API version 1.21 corresponds to Docker 1.9, which is currently the
 	// minimum version we want to support.
@@ -1647,11 +1652,6 @@ func main() {
 
 	cr := NewContainerRunner(api, kc, dockerClientProxy, containerId)
 
-	if kcerr != nil {
-		cr.CrunchLog.Printf("%s: %v", containerId, kcerr)
-		cr.CrunchLog.Close()
-		os.Exit(1)
-	}
 	if dockererr != nil {
 		cr.CrunchLog.Printf("%s: %v", containerId, dockererr)
 		cr.checkBrokenNode(dockererr)
@@ -1659,8 +1659,6 @@ func main() {
 		os.Exit(1)
 	}
 
-	kc.BlockCache = &keepclient.BlockCache{MaxBlocks: 2}
-	kc.Retries = 4
 	cr.statInterval = *statInterval
 	cr.cgroupRoot = *cgroupRoot
 	cr.expectCgroupParent = *cgroupParent

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list