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

Queued status for pipelines not reported correctly

    XMLWordPrintable

    Details

    • Similar Issues:
    • Epic Link:
    • Sprint:
      Blue Ocean 1.2-beta1, Blue Ocean 1.2-beta2, Blue Ocean 1.2-beta3, Blue Ocean 1.2-beta4, Blue Ocean 1.2

      Description

      Problem
      Pipelines that are queued are reported as running

      Steps to reproduce

      1. Setup the app store demo pipeline https://github.com/i386/app-store-demo
      2. disable all executors
      3. run the master branch
      4. The run is reported in Blue Ocean as running rather than queued

      Notes
      Pipeline queue item is queued but the run is reported as running BlueRunState.RUNNING when it should be reported as BlueRunState.QUEUED

      If no nodes in pipeline are running then run state is QUEUED as it is indeed. If at least one of the node is in running state Run state is RUNNING.

      Its important to know that we do some things at the Pipeline run level to calculate the "input" state of a Pipeline run in Blue Ocean (I think we search for the InputAction on the run and if there are any then we say that the state is "input")

        Attachments

        1. blue.png
          blue.png
          205 kB
        2. queue.png
          queue.png
          45 kB

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. JENKINS-44981 Developer can see node steps within the step listing

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-41344 Steps with "causeOfBlockage" should show indeterminate indicator

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open
          links to

          Web Link Blue Ocean PR #1184

            Activity

            Ascending order - Click to sort in descending order
            4 older comments
            Hide
            Permalink
            abayer Andrew Bayer added a comment -

            PR up at https://github.com/jenkinsci/blueocean-plugin/pull/1184 - originally thought of it as JENKINS-44981, but I've realized that it should actually be solving this issue directly, while it'll take some more UI-side changes to solve JENKINS-44981.

            Show
            abayer Andrew Bayer added a comment - PR up at https://github.com/jenkinsci/blueocean-plugin/pull/1184 - originally thought of it as JENKINS-44981 , but I've realized that it should actually be solving this issue directly, while it'll take some more UI-side changes to solve JENKINS-44981 .
            Hide
            Permalink
            abayer Andrew Bayer added a comment -

            James Dumay So I don't actually know what a queued run should look like. =)

            Show
            abayer Andrew Bayer added a comment - James Dumay So I don't actually know what a queued run should look like. =)
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Andrew Bayer
            Path:
            src/main/java/org/jenkinsci/plugins/workflow/actions/QueueItemAction.java
            http://jenkins-ci.org/commit/workflow-api-plugin/41c239f904847f9a7fbdd46fc3502e35c21cdb19
            Log:
            Merge pull request #42 from abayer/jenkins-44981

            JENKINS-44636 Action for tracking node block queued/running status

            Compare: https://github.com/jenkinsci/workflow-api-plugin/compare/02c14940016a...41c239f90484

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Bayer Path: src/main/java/org/jenkinsci/plugins/workflow/actions/QueueItemAction.java http://jenkins-ci.org/commit/workflow-api-plugin/41c239f904847f9a7fbdd46fc3502e35c21cdb19 Log: Merge pull request #42 from abayer/jenkins-44981 JENKINS-44636 Action for tracking node block queued/running status Compare: https://github.com/jenkinsci/workflow-api-plugin/compare/02c14940016a...41c239f90484
            Hide
            Permalink
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Sam Van Oort
            Path:
            pom.xml
            src/main/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/GenericStatus.java
            src/main/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/StatusAndTiming.java
            src/test/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/StatusAndTimingTest.java
            http://jenkins-ci.org/commit/pipeline-graph-analysis-plugin/42960f027a0685b26cbb9cf37034c16358777ad1
            Log:
            Merge pull request #11 from abayer/jenkins-44981

            JENKINS-44636 Add GenericStatus.QUEUED and related logic

            Compare: https://github.com/jenkinsci/pipeline-graph-analysis-plugin/compare/b18e165c0ac8...42960f027a06

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sam Van Oort Path: pom.xml src/main/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/GenericStatus.java src/main/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/StatusAndTiming.java src/test/java/org/jenkinsci/plugins/workflow/pipelinegraphanalysis/StatusAndTimingTest.java http://jenkins-ci.org/commit/pipeline-graph-analysis-plugin/42960f027a0685b26cbb9cf37034c16358777ad1 Log: Merge pull request #11 from abayer/jenkins-44981 JENKINS-44636 Add GenericStatus.QUEUED and related logic Compare: https://github.com/jenkinsci/pipeline-graph-analysis-plugin/compare/b18e165c0ac8...42960f027a06
            Hide
            Permalink
            jamesdumay James Dumay added a comment -

            Andrew Bayer I need to do some testing of this today to see if this goes into 1.2 or not. Will report back on the PR.

            Show
            jamesdumay James Dumay added a comment - Andrew Bayer I need to do some testing of this today to see if this goes into 1.2 or not. Will report back on the PR.

              People

              Assignee:
              abayer Andrew Bayer
              Reporter:
              jamesdumay James Dumay
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: