[ARVADOS] updated: 88ce5e8a1fe79c696bd78ff3556193fc0436a6e5

git at public.curoverse.com git at public.curoverse.com
Tue Feb 9 16:25:49 EST 2016


Summary of changes:
 services/api/lib/crunch_dispatch.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  88ce5e8a1fe79c696bd78ff3556193fc0436a6e5 (commit)
      from  427c7d7de5017e322c5bf359ea878c9e7b406337 (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 88ce5e8a1fe79c696bd78ff3556193fc0436a6e5
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Tue Feb 9 16:25:45 2016 -0500

    8406: @job_retry_counts.include? jobrecord.uuid because @job_retry_counts has a default value.

diff --git a/services/api/lib/crunch_dispatch.rb b/services/api/lib/crunch_dispatch.rb
index 06a8a4b..131386d 100644
--- a/services/api/lib/crunch_dispatch.rb
+++ b/services/api/lib/crunch_dispatch.rb
@@ -637,7 +637,7 @@ class CrunchDispatch
 
     jobrecord = Job.find_by_uuid(job_done.uuid)
 
-    if exit_status == EXIT_RETRY_UNLOCKED or (exit_tempfail and @job_retry_counts[jobrecord.uuid])
+    if exit_status == EXIT_RETRY_UNLOCKED or (exit_tempfail and @job_retry_counts.include? jobrecord.uuid)
       # The job failed because all of the nodes allocated to it
       # failed.  Only this crunch-dispatch process can retry the job:
       # it's already locked, and there's no way to put it back in the

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list