Project

General

Profile

Actions

Bug #10782

closed

[Crunch2] Retrying request reassigns all requests

Added by Peter Amstutz over 5 years ago. Updated over 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 1 (0 open1 closed)

Task #10783: Review 10782-container-retryResolvedLucas Di Pentima12/28/2016

Actions
Actions #1

Updated by Peter Amstutz over 5 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz over 5 years ago

  • Assigned To set to Peter Amstutz
Actions #3

Updated by Lucas Di Pentima over 5 years ago

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

Actions #4

Updated by Peter Amstutz over 5 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:997479d1408139e96ecdb42a60b4f727f814f6c9.

Actions

Also available in: Atom PDF