[ARVADOS] updated: f927cd24852edcd7a8389533e96a16cf6877e58f

Git user git at public.curoverse.com
Wed Feb 1 11:47:31 EST 2017


Summary of changes:
 services/crunch-dispatch-slurm/crunch-dispatch-slurm.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

       via  f927cd24852edcd7a8389533e96a16cf6877e58f (commit)
       via  7d29578411f1940ef5cc5ce985766dd81d133aaa (commit)
      from  c3d4f8a585202ec58df5506934b698039c200b68 (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 f927cd24852edcd7a8389533e96a16cf6877e58f
Merge: c3d4f8a 7d29578
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Feb 1 11:46:26 2017 -0500

    Merge branch '10705-dedup-log-message'
    
    refs #10705


commit 7d29578411f1940ef5cc5ce985766dd81d133aaa
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Feb 1 11:38:29 2017 -0500

    10705: Fix double-logged error.

diff --git a/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go b/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
index 1c080f3..e8c0a27 100644
--- a/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
+++ b/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
@@ -160,10 +160,7 @@ func submit(dispatcher *dispatch.Dispatcher,
 			// OK, no cleanup needed
 			return
 		}
-		err := dispatcher.Unlock(container.UUID)
-		if err != nil {
-			log.Printf("Error unlocking container %s: %v", container.UUID, err)
-		}
+		dispatcher.Unlock(container.UUID)
 	}()
 
 	cmd := sbatchCmd(container)

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list