Bug #4046

[API] All jobs claim to be "next in queue". Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index.

Added by Radhika Chippada about 5 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
-
Target version:
Start date:
10/06/2014
Due date:
% Done:

100%

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

Subtasks

Task #4117: Review branch: 4046-job-queue-positionResolvedRadhika Chippada

Associated revisions

Revision 3b572b68
Added by Radhika Chippada about 5 years ago

closes #4046
Merge branch '4046-job-queue-position'

History

#1 Updated by Radhika Chippada about 5 years ago

  • Subject changed from Bug in queue_position calculation services/api/app/models/job.rb. It needs to increment index. to Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index.

#2 Updated by Tom Clegg about 5 years ago

  • Story points set to 0.5

#3 Updated by Tom Clegg about 5 years ago

  • Target version changed from Bug Triage to Arvados Future Sprints

#4 Updated by Tom Clegg about 5 years ago

  • Subject changed from Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index. to [API] Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index.

#5 Updated by Tom Clegg about 5 years ago

  • Subject changed from [API] Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index. to [API] All jobs claim to be "next in queue". Bug in queue_position calculation in services/api/app/models/job.rb. It needs to increment index.

#6 Updated by Radhika Chippada about 5 years ago

  • Status changed from New to In Progress
  • Assigned To set to Radhika Chippada

#7 Updated by Radhika Chippada about 5 years ago

  • Target version changed from Arvados Future Sprints to 2014-10-08 sprint

#8 Updated by Brett Smith about 5 years ago

Reviewing bbec601

  • Please make the job.state assert messages distinct, to reduce the risk that someone might see an error from one but look at the other assertion in the test.
  • Please clean up the trailing whitespace on the blank line after the new test.

With these clean-ups, this is good to merge. Thanks.

#9 Updated by Radhika Chippada about 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:3b572b689b2513e193f77d78fb3b58b1fc0a6700.

Also available in: Atom PDF