Project

General

Profile

Actions

Bug #4046

closed

[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 over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
-
Target version:
Story points:
0.5

Subtasks 1 (0 open1 closed)

Task #4117: Review branch: 4046-job-queue-positionResolvedRadhika Chippada10/06/2014Actions
Actions #1

Updated by Radhika Chippada over 9 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.
Actions #2

Updated by Tom Clegg over 9 years ago

  • Story points set to 0.5
Actions #3

Updated by Tom Clegg over 9 years ago

  • Target version changed from Bug Triage to Arvados Future Sprints
Actions #4

Updated by Tom Clegg over 9 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.
Actions #5

Updated by Tom Clegg over 9 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.
Actions #6

Updated by Radhika Chippada over 9 years ago

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

Updated by Radhika Chippada over 9 years ago

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

Updated by Brett Smith over 9 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.

Actions #9

Updated by Radhika Chippada over 9 years ago

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

Applied in changeset arvados|commit:3b572b689b2513e193f77d78fb3b58b1fc0a6700.

Actions

Also available in: Atom PDF