8406: @job_retry_counts.include? jobrecord.uuid because @job_retry_counts has a defau...
authorPeter Amstutz <peter.amstutz@curoverse.com>
Tue, 9 Feb 2016 21:25:45 +0000 (16:25 -0500)
committerBrett Smith <brett@curoverse.com>
Tue, 9 Feb 2016 21:42:37 +0000 (16:42 -0500)
services/api/lib/crunch_dispatch.rb

index 06a8a4b633b00353d4d96f099fcff97ef610bab6..131386d765f66b66d958d244a6590131d0189769 100644 (file)
@@ -637,7 +637,7 @@ class CrunchDispatch
 
     jobrecord = Job.find_by_uuid(job_done.uuid)
 
 
     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
       # 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