Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-27034

BuildableItems (Subtasks) are miscounted in the queue

      BuildableItems (Subtasks) are miscounted in the queue

      Expected:
      When a labeled job is in the queue its nodeprovisioner should report excess load and provision a new node. The "free roaming" nodeprovisioner should report no load and not provision a new node.

      Actual:
      In addition to JENKINS-25207 (The free roaming nodeprovisioner spawns nodes even though the job is labeled), it is possible for the appropriate nodeprovisioner to not report load because of use of == on the label.

      Technical:
      Labels are compared in a number of places with == instead of equals

      Additionally af55e5345628d31f49390e11a473079bed750fd9, modified the behaviour of the "free roaming" label in the queue. This results in spawning nodes for "free roaming" jobs, even when none exist.

      I have included a pull request that resolves these issues.

          [JENKINS-27034] BuildableItems (Subtasks) are miscounted in the queue

          Thomas Suckow added a comment -

          The pull request in this issue should resolve spawning non-exclusive nodes when only labeled jobs are in the queue.

          Thomas Suckow added a comment - The pull request in this issue should resolve spawning non-exclusive nodes when only labeled jobs are in the queue.

          Thomas Suckow added a comment -

          Upping to Major because I have to rebuild Jenkins for each update and I don't really have the time to keep doing that for every update.

          Thomas Suckow added a comment - Upping to Major because I have to rebuild Jenkins for each update and I don't really have the time to keep doing that for every update.

          Upping to critical as description looks very obvious and i think some bug reports to cloud plugins depends on it.

          Kanstantsin Shautsou added a comment - Upping to critical as description looks very obvious and i think some bug reports to cloud plugins depends on it.

            Unassigned Unassigned
            thomassuckow Thomas Suckow
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: