[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