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

ExecutorStep::labels should be recorded in the FlowNode graph

XMLWordPrintable

      In the current flow graph it's impossible to understand, which labels have been actually required by the node() call. Such info would be useful for the build analysis.

      In WorkspaceActionImpl we get all labels of the build, so this info is not relevant.

          [JENKINS-33995] ExecutorStep::labels should be recorded in the FlowNode graph

          Oleg Nenashev created issue -
          Oleg Nenashev made changes -
          Description Original: In the current flow graph it's impossible to understand, which labels have been actually required by the node() call. Such info would be useful for the build analysis.

          In WorkspaceActionImpl we get all labels of the build, so this info is not relevant           		New: In the current flow graph it's impossible to understand, which labels have been actually required by the node() call. Such info would be useful for the build analysis.

          In WorkspaceActionImpl we get all labels of the build, so this info is not relevant.

          Proposal:
          * Change the behavior of WorkspaceActionImpl#labels , because it it not documented
          * WorkspaceActionImpl#labels should list labels, which have been actually required by the run
          * WorkspaceActionImpl#allNodeLabels should store all node labels
          Jesse Glick made changes -
          Labels New: api
          Jesse Glick made changes -
          Link New: This issue is related to JENKINS-31582 [ JENKINS-31582 ]
          Jesse Glick made changes -
          Link New: This issue is related to JENKINS-32225 [ JENKINS-32225 ]
          Oleg Nenashev made changes -
          Description Original: In the current flow graph it's impossible to understand, which labels have been actually required by the node() call. Such info would be useful for the build analysis.

          In WorkspaceActionImpl we get all labels of the build, so this info is not relevant.

          Proposal:
          * Change the behavior of WorkspaceActionImpl#labels , because it it not documented
          * WorkspaceActionImpl#labels should list labels, which have been actually required by the run
          * WorkspaceActionImpl#allNodeLabels should store all node labels           		New: In the current flow graph it's impossible to understand, which labels have been actually required by the node() call. Such info would be useful for the build analysis.

          In WorkspaceActionImpl we get all labels of the build, so this info is not relevant.
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 170010 ] New: JNJira + In-Review [ 183726 ]
          Andrew Bayer made changes -
          Component/s New: pipeline-general [ 21692 ]
          Andrew Bayer made changes -
          Component/s Original: workflow-plugin [ 18820 ]
          Jesse Glick made changes -
          Component/s New: workflow-durable-task-step-plugin [ 21715 ]
          Component/s Original: pipeline [ 21692 ]
          Oleg Nenashev made changes -
          Link New: This issue is duplicated by JENKINS-52394 [ JENKINS-52394 ]

            jglick Jesse Glick
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: