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

BuildableItems (Subtasks) are miscounted in the queue

XMLWordPrintable

      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 created issue -
          Thomas Suckow made changes -
          Environment New: Jenkins 1.598

          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 made changes -
          Link New: This issue is related to JENKINS-25207 [ JENKINS-25207 ]

          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.
          Thomas Suckow made changes -
          Priority Original: Minor [ 4 ] New: Major [ 3 ]
          Kanstantsin Shautsou made changes -
          Priority Original: Major [ 3 ] New: Critical [ 2 ]

          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.

          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.
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 161225 ] New: JNJira + In-Review [ 180617 ]

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

              Created:
              Updated: