• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • pipeline

      Adjust how steps are displayed, to make the appearance cleaner and keep a focus on the DSL. Suggested changes:

      • Make Snippet Generator list the StepDescriptor.functionName first, as a sort key, followed by the StepDescriptor.displayName. So for example, you might see a pulldown entry sh: run a Unix shell script.
      • Change the console log to print just the functionName, not the displayName.
        • For steps which add LabelAction, like stage, follow the function name with the label: stage: Production. That probably means this call is no longer useful.
        • For block start/stop nodes, perhaps show something like node { or } // node.
        • Metasteps (checkout, step, wrap) need changes but this is covered by JENKINS-29922.
        • Ideally, use ConsoleNote to render metadata with special markup so it does not distract from build output.
      • Tweak the display of Workflow Steps similarly. (FlowNode.getDisplayName might still be OK here if the implementation were modified somewhat.)

          [JENKINS-30088] Clean up step display

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java
          http://jenkins-ci.org/commit/workflow-api-plugin/4f0f99d7e51f4ea2337e19c1981d71c138cb6419
          Log:
          JENKINS-30088 Use the function name instead of the display name

          Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java http://jenkins-ci.org/commit/workflow-api-plugin/4f0f99d7e51f4ea2337e19c1981d71c138cb6419 Log: JENKINS-30088 Use the function name instead of the display name Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java
          http://jenkins-ci.org/commit/workflow-api-plugin/c39cdeef49ea353601df1383efcc61f28247f5d0
          Log:
          JENKINS-30088 Improve javadoc and use getDisplayName() as default

          Originally-Committed-As: dcaf836efe64e46df36d04a022478396231eddcd

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java http://jenkins-ci.org/commit/workflow-api-plugin/c39cdeef49ea353601df1383efcc61f28247f5d0 Log: JENKINS-30088 Improve javadoc and use getDisplayName() as default Originally-Committed-As: dcaf836efe64e46df36d04a022478396231eddcd

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java
          http://jenkins-ci.org/commit/workflow-api-plugin/95bf764f7e0cd6b64e7a99086226db1dd8ca2b35
          Log:
          Light changes to console logs forgotten in JENKINS-30088 fix

          Originally-Committed-As: 066762028c673b6fd262d5afbcff1321a054aaab

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: api/src/main/java/org/jenkinsci/plugins/workflow/graph/FlowNode.java http://jenkins-ci.org/commit/workflow-api-plugin/95bf764f7e0cd6b64e7a99086226db1dd8ca2b35 Log: Light changes to console logs forgotten in JENKINS-30088 fix Originally-Committed-As: 066762028c673b6fd262d5afbcff1321a054aaab

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepAtomNode.java
          http://jenkins-ci.org/commit/workflow-cps-plugin/536f73a7b2e68e8f12e8d20ca8973e528d556025
          Log:
          JENKINS-30088 Use the function name instead of the display name

          Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepAtomNode.java http://jenkins-ci.org/commit/workflow-cps-plugin/536f73a7b2e68e8f12e8d20ca8973e528d556025 Log: JENKINS-30088 Use the function name instead of the display name Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java
          http://jenkins-ci.org/commit/workflow-cps-plugin/18f2f4319c3b3a53c0b4d5942c0674ea578d02ac
          Log:
          JENKINS-30088 Annotate workflow metadata logs

          Originally-Committed-As: 7e2c3a0f52684f989b5c398a9ffd99739a8f7b14

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java http://jenkins-ci.org/commit/workflow-cps-plugin/18f2f4319c3b3a53c0b4d5942c0674ea578d02ac Log: JENKINS-30088 Annotate workflow metadata logs Originally-Committed-As: 7e2c3a0f52684f989b5c398a9ffd99739a8f7b14

          Code changed in jenkins
          User: Jesse Glick
          Path:
          aggregator/src/test/java/org/jenkinsci/plugins/workflow/WorkflowJobNonRestartingTest.java
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepAtomNode.java
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java
          http://jenkins-ci.org/commit/workflow-cps-plugin/e955d74f926ae45bc27c619646bf9d33adbe41fa
          Log:
          Merge pull request #215 from amuniz/JENKINS-30088

          JENKINS-30088 Clean up step display in logs
          Originally-Committed-As: 9ae86c4219d741e1e6e5a99ef1df27fdf08edb5c

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: aggregator/src/test/java/org/jenkinsci/plugins/workflow/WorkflowJobNonRestartingTest.java cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepAtomNode.java cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java cps/src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java http://jenkins-ci.org/commit/workflow-cps-plugin/e955d74f926ae45bc27c619646bf9d33adbe41fa Log: Merge pull request #215 from amuniz/ JENKINS-30088 JENKINS-30088 Clean up step display in logs Originally-Committed-As: 9ae86c4219d741e1e6e5a99ef1df27fdf08edb5c

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          job/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java
          job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowRunConsoleNote.java
          http://jenkins-ci.org/commit/workflow-job-plugin/d10e6ae4287066d3868fca1af4e42deae7ea47b0
          Log:
          JENKINS-30088 Use the function name instead of the display name

          Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: job/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowRunConsoleNote.java http://jenkins-ci.org/commit/workflow-job-plugin/d10e6ae4287066d3868fca1af4e42deae7ea47b0 Log: JENKINS-30088 Use the function name instead of the display name Originally-Committed-As: 577f2acb256c0c91eaa81abe3177b4248bbaa059

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          job/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java
          job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowConsoleLogger.java
          job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowMetadataConsoleFilter.java
          job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowRunConsoleNote.java
          http://jenkins-ci.org/commit/workflow-job-plugin/99afbdb0f16a0cb49e153d1d9ff157f68476c0ce
          Log:
          JENKINS-30088 Annotate workflow metadata logs

          Originally-Committed-As: 7e2c3a0f52684f989b5c398a9ffd99739a8f7b14

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: job/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowConsoleLogger.java job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowMetadataConsoleFilter.java job/src/main/java/org/jenkinsci/plugins/workflow/job/console/WorkflowRunConsoleNote.java http://jenkins-ci.org/commit/workflow-job-plugin/99afbdb0f16a0cb49e153d1d9ff157f68476c0ce Log: JENKINS-30088 Annotate workflow metadata logs Originally-Committed-As: 7e2c3a0f52684f989b5c398a9ffd99739a8f7b14

          Code changed in jenkins
          User: Antonio Muñiz
          Path:
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/steps/LoadStepExecution.java
          cps/src/main/java/org/jenkinsci/plugins/workflow/cps/steps/ParallelStepExecution.java
          http://jenkins-ci.org/commit/workflow-cps-plugin/de78cd713a6797db9e6144227da94ee234c6d3ad
          Log:
          Light changes to console logs forgotten in JENKINS-30088 fix

          Originally-Committed-As: 066762028c673b6fd262d5afbcff1321a054aaab

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Antonio Muñiz Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/steps/LoadStepExecution.java cps/src/main/java/org/jenkinsci/plugins/workflow/cps/steps/ParallelStepExecution.java http://jenkins-ci.org/commit/workflow-cps-plugin/de78cd713a6797db9e6144227da94ee234c6d3ad Log: Light changes to console logs forgotten in JENKINS-30088 fix Originally-Committed-As: 066762028c673b6fd262d5afbcff1321a054aaab

          Code changed in jenkins
          User: Jesse Glick
          Path:
          src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java
          src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java
          src/test/java/org/jenkinsci/plugins/workflow/WorkflowJobNonRestartingTest.java
          http://jenkins-ci.org/commit/workflow-cps-plugin/2ef59e10507b5293912913f4865b26ead5568d02
          Log:
          JENKINS-30088 Cleaner appearance of nodes with blocks.
          · Use the function name, not step display name, on the overall start and end nodes.
          · Omit any text on the block start and end nodes.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepEndNode.java src/main/java/org/jenkinsci/plugins/workflow/cps/nodes/StepStartNode.java src/test/java/org/jenkinsci/plugins/workflow/WorkflowJobNonRestartingTest.java http://jenkins-ci.org/commit/workflow-cps-plugin/2ef59e10507b5293912913f4865b26ead5568d02 Log: JENKINS-30088 Cleaner appearance of nodes with blocks. · Use the function name, not step display name, on the overall start and end nodes. · Omit any text on the block start and end nodes.

            amuniz Antonio Muñiz
            jglick Jesse Glick
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: