• Custom Tools Compatibility

      Add support for workflow

      Acceptance criteria from the plugin maintainer (oleg_nenashev

      • UsedCustomToolsAction RunAction2, which tracks the list of required tools
      • "Install custom tool" simple build step, which adds the tool to the scope of the task (AbstractProject or Pipeline's node() scope). Stores data in UsedCustomToolsAction. Pipeline alias - e.g. customTool()
      • Ability to contribute environment variables for launched scripts, likely with a new LauncherDecorator method (decorate(Launcher launcher, Job job, Node node) or so) - depends on JENKINS-TODO, can be done in a compatible way
      • Update the DecoratedLauncher implementation and to move it outside the BuildWrapper to the new LauncherDecorator
      • New DecoratedLauncher implementation pulls information about tools from UsedCustomToolsAction and then initializes it, similarly to what it does now with pulling from the Build Wrapper
      • Automatic test(s) for the common use-cases in Pipeline (see the requirements below)
      • Documentation, can be written in a new page within the GitHub repository (Wiki must die soon). This documentation should describe the logic and provide examples.

      Extra requirements:

      • The version engine is supported including the Tool Version parameter. Should be no problems with that, but needs testing
      • The resolution of variables in the step takes other Node and Job environment variables into account

      Nice2have, optional:

      • Some UI, which improves tracking of tool usage in UsedCustomToolsAction - really important for multi-log Pipeline
      • Support definition of tools on the global level (for all node() calls)
      • CustomToolsBuildWrapper is reworked to the SimpleBuildWrapper
        • oleg_nenashev: Not really required if there is a buildStep described above
      • Common tool() step is able to invoke the customTool() logic somehow.

       

          [JENKINS-30680] Make Custom Tools compatible with Pipeline

          Nigel Harniman created issue -

          Code changed in jenkins
          User: Nigel Harniman
          Path:
          COMPATIBILITY.md
          http://jenkins-ci.org/commit/workflow-plugin/0c58bff762739ece6bbcc25c4fb4d02ea198eee3
          Log:
          Noting JENKINS-30680

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nigel Harniman Path: COMPATIBILITY.md http://jenkins-ci.org/commit/workflow-plugin/0c58bff762739ece6bbcc25c4fb4d02ea198eee3 Log: Noting JENKINS-30680

          Code changed in jenkins
          User: Nigel Harniman
          Path:
          COMPATIBILITY.md
          TUTORIAL.md
          http://jenkins-ci.org/commit/workflow-plugin/78a20d902c6ac5c069cfc4f38bdade36e109cc4c
          Log:
          Noting JENKINS-30680

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nigel Harniman Path: COMPATIBILITY.md TUTORIAL.md http://jenkins-ci.org/commit/workflow-plugin/78a20d902c6ac5c069cfc4f38bdade36e109cc4c Log: Noting JENKINS-30680

          Code changed in jenkins
          User: Nigel Harniman
          Path:
          COMPATIBILITY.md
          http://jenkins-ci.org/commit/workflow-plugin/5ebcdcb06c61faf2e0c4e8dc0c2fbdf7ede70267
          Log:
          Noting JENKINS-30680

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nigel Harniman Path: COMPATIBILITY.md http://jenkins-ci.org/commit/workflow-plugin/5ebcdcb06c61faf2e0c4e8dc0c2fbdf7ede70267 Log: Noting JENKINS-30680

          Code changed in jenkins
          User: Jesse Glick
          Path:
          COMPATIBILITY.md
          http://jenkins-ci.org/commit/workflow-plugin/9d94b720d9f225d4f0f82d61e8f7599dbdabaa1f
          Log:
          Merge pull request #214 from harniman/master

          Noting JENKINS-30680

          Compare: https://github.com/jenkinsci/workflow-plugin/compare/47f0fb752d20...9d94b720d9f2

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: COMPATIBILITY.md http://jenkins-ci.org/commit/workflow-plugin/9d94b720d9f225d4f0f82d61e8f7599dbdabaa1f Log: Merge pull request #214 from harniman/master Noting JENKINS-30680 Compare: https://github.com/jenkinsci/workflow-plugin/compare/47f0fb752d20...9d94b720d9f2
          Patrick Wolf made changes -
          Epic Link New: JENKINS-34657 [ 170293 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 165797 ] New: JNJira + In-Review [ 182148 ]
          Oleg Nenashev made changes -
          Link New: This issue is duplicated by JENKINS-37413 [ JENKINS-37413 ]
          Andrew Bayer made changes -
          Labels Original: workflow New: pipeline workflow
          Andrew Bayer made changes -
          Labels Original: pipeline workflow New: pipeline

            Unassigned Unassigned
            nharniman Nigel Harniman
            Votes:
            21 Vote for this issue
            Watchers:
            32 Start watching this issue

              Created:
              Updated: