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

SimpleBuildWrapper

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • pipeline

    Description

      This may be something that I've missed but how are you planing on adding support to add build wrappers to the Workflow plugin. We depend a lot on a few plugins to modify the console output to for example color code, find match patterns and to filter out password.

      Attachments

        Issue Links

          Activity

            jglick Jesse Glick added a comment -

            Build wrappers could perhaps be integrated by defining a SimpleBuildWrapper API (analogous to SimpleBuildStep and so taking Run and FilePath rather than AbstractBuild) where Environment is considered Serializable, and introducing a generic Step to run these. This could make it easier to use xvnc-plugin etc.

            The tool-env plugin does not need this, due to the standard tool step.

            jglick Jesse Glick added a comment - Build wrappers could perhaps be integrated by defining a SimpleBuildWrapper API (analogous to SimpleBuildStep and so taking Run and FilePath rather than AbstractBuild ) where Environment is considered Serializable , and introducing a generic Step to run these. This could make it easier to use xvnc-plugin etc. The tool-env plugin does not need this, due to the standard tool step.
            jglick Jesse Glick added a comment -

            Went ahead and wrote a custom Step for the Credentials Binding plugin, though it could perhaps have been written using the API under development.

            Not currently attempting to handling wrappers which decorate the console, as I do not see any straightforward way to support this transparently. If you have some specific plugins in mind, please list the most important ones so I can study what they do and whether that can possibly make sense from within a workflow.

            jglick Jesse Glick added a comment - Went ahead and wrote a custom Step for the Credentials Binding plugin, though it could perhaps have been written using the API under development. Not currently attempting to handling wrappers which decorate the console, as I do not see any straightforward way to support this transparently. If you have some specific plugins in mind, please list the most important ones so I can study what they do and whether that can possibly make sense from within a workflow.

            Code changed in jenkins
            User: Jesse Glick
            Path:
            COMPATIBILITY.md
            http://jenkins-ci.org/commit/workflow-plugin/f3f8cc06f50be32ae54783c959ea916f803dfa83
            Log:
            Updates related to JENKINS-24673

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: COMPATIBILITY.md http://jenkins-ci.org/commit/workflow-plugin/f3f8cc06f50be32ae54783c959ea916f803dfa83 Log: Updates related to JENKINS-24673

            Code changed in jenkins
            User: Jesse Glick
            Path:
            changelog.html
            core/src/main/java/jenkins/tasks/SimpleBuildWrapper.java
            test/src/test/java/jenkins/tasks/SimpleBuildWrapperTest.java
            http://jenkins-ci.org/commit/jenkins/416bdef94db123ed8d051d095007491c9184b6ce
            Log:
            JENKINS-24673 Noting merge of #1530.

            Compare: https://github.com/jenkinsci/jenkins/compare/1dd60ec77808...416bdef94db1

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: changelog.html core/src/main/java/jenkins/tasks/SimpleBuildWrapper.java test/src/test/java/jenkins/tasks/SimpleBuildWrapperTest.java http://jenkins-ci.org/commit/jenkins/416bdef94db123ed8d051d095007491c9184b6ce Log: JENKINS-24673 Noting merge of #1530. Compare: https://github.com/jenkinsci/jenkins/compare/1dd60ec77808...416bdef94db1
            jglick Jesse Glick added a comment -

            Core change merged towards 1.599.

            jglick Jesse Glick added a comment - Core change merged towards 1.599.

            Was looking a bit more on this now that I got some free time again, the only thing we need right now is to be able to create a wrapper for the listener output. So we can filter the output and color code the output. For example we have a plugin that formats or output simular to https://wiki.jenkins-ci.org/display/JENKINS/Timestamper and https://wiki.jenkins-ci.org/display/JENKINS/AnsiColor+Plugin as well as a plugin to filer out passwords from the console.

            hanabishi Marcus Jacobsson added a comment - Was looking a bit more on this now that I got some free time again, the only thing we need right now is to be able to create a wrapper for the listener output. So we can filter the output and color code the output. For example we have a plugin that formats or output simular to https://wiki.jenkins-ci.org/display/JENKINS/Timestamper and https://wiki.jenkins-ci.org/display/JENKINS/AnsiColor+Plugin as well as a plugin to filer out passwords from the console.
            jglick Jesse Glick added a comment - hanabishi see JENKINS-27392 .
            jglick Jesse Glick added a comment -

            Workflow 1.8 has the new wrap step.

            jglick Jesse Glick added a comment - Workflow 1.8 has the new wrap step.

            People

              jglick Jesse Glick
              hanabishi Marcus Jacobsson
              Votes:
              3 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: