Bug #10782

[Crunch2] Retrying request reassigns all requests

Added by Peter Amstutz almost 5 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
12/28/2016
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

In API server container.rb:

retryable_requests = ContainerRequest.where("priority > 0 and state = 'Committed' and container_count < container_count_max")

It's not filtering on container_uuid which means it reassigns every outstanding container request and not just the ones associated with container being retried.


Subtasks

Task #10783: Review 10782-container-retryResolvedLucas Di Pentima

Associated revisions

Revision 997479d1
Added by Peter Amstutz almost 5 years ago

Merge branch '10782-container-retry' closes #10782

History

#1 Updated by Peter Amstutz almost 5 years ago

  • Description updated (diff)

#2 Updated by Peter Amstutz almost 5 years ago

  • Assigned To set to Peter Amstutz

#3 Updated by Lucas Di Pentima almost 5 years ago

LGTM. I ran the modified test file successfully on my local machine.

#4 Updated by Peter Amstutz almost 5 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:997479d1408139e96ecdb42a60b4f727f814f6c9.

Also available in: Atom PDF