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

Enable use of managed files for templates

    XMLWordPrintable

Details

    Description

      At the moment in order to use a content template the script must be copied into the folder $JENKINS_ROOT/email-templates, which requires filesystem access to the master.
      As an alternative I want to be able to create a managed file (using the Config File Provider plugin) and then reference that file when configuring editable email notification build step.

      Attachments

        Activity

          tyrius Jonathan Zimmerman created issue -
          slide_o_mix Alex Earl made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]

          Code changed in jenkins
          User: Alex Earl
          Path:
          pom.xml
          src/main/java/hudson/plugins/emailext/JellyTemplateConfig.java
          src/main/java/hudson/plugins/emailext/plugins/content/JellyScriptContent.java
          src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/JellyTemplateConfigProvider/newInstanceDetail.jelly
          src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/JellyTemplateConfigProvider/newInstanceDetail.properties
          src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/edit-config.jelly
          http://jenkins-ci.org/commit/email-ext-plugin/9c8e0ac16c440293728a5f7343993857c5f47b00
          Log:
          Fix JENKINS-18203

          Implement support for managed files as templates.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Alex Earl Path: pom.xml src/main/java/hudson/plugins/emailext/JellyTemplateConfig.java src/main/java/hudson/plugins/emailext/plugins/content/JellyScriptContent.java src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/JellyTemplateConfigProvider/newInstanceDetail.jelly src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/JellyTemplateConfigProvider/newInstanceDetail.properties src/main/resources/hudson/plugins/emailext/JellyTemplateConfig/edit-config.jelly http://jenkins-ci.org/commit/email-ext-plugin/9c8e0ac16c440293728a5f7343993857c5f47b00 Log: Fix JENKINS-18203 Implement support for managed files as templates.
          slide_o_mix Alex Earl added a comment -

          This is still a work in progress...didn't think about the commit message all the way apparently.

          slide_o_mix Alex Earl added a comment - This is still a work in progress...didn't think about the commit message all the way apparently.
          slide_o_mix Alex Earl added a comment -

          Added ability to use managed files for both Groovy and Jelly templates. To use the template you specify it like this:

          ${JELLY_SCRIPT, template="managed:TEMPLATE NAME"}
          ${SCRIPT, template="managed:TEMPLATE NAME"}

          Note, it is the NAME of the config item, not the ID. Two new types of managed files are available in the "managed config file" area for each of the above content tokens.

          slide_o_mix Alex Earl added a comment - Added ability to use managed files for both Groovy and Jelly templates. To use the template you specify it like this: ${JELLY_SCRIPT, template="managed:TEMPLATE NAME"} ${SCRIPT, template="managed:TEMPLATE NAME"} Note, it is the NAME of the config item, not the ID. Two new types of managed files are available in the "managed config file" area for each of the above content tokens.
          slide_o_mix Alex Earl made changes -
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Resolved [ 5 ]

          Can you update the token help to include the "managed:" syntax?

          walterk82 Walter Kacynski added a comment - Can you update the token help to include the "managed:" syntax?
          walterk82 Walter Kacynski made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          slide_o_mix Alex Earl added a comment -

          Good idea, I'll do that.

          slide_o_mix Alex Earl added a comment - Good idea, I'll do that.
          slide_o_mix Alex Earl added a comment -

          Added information on the managed: syntax to the help for the SCRIPT and JELLY_SCRIPT tokens.

          slide_o_mix Alex Earl added a comment - Added information on the managed: syntax to the help for the SCRIPT and JELLY_SCRIPT tokens.
          slide_o_mix Alex Earl made changes -
          Resolution Fixed [ 1 ]
          Status Reopened [ 4 ] Resolved [ 5 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 149501 ] JNJira + In-Review [ 193153 ]
          cimbi Martin Cimbalek added a comment - - edited

          Hello guys. Thank you for this feature. However, I am not able to get the "managed:" feature work. It simply cannot find the file (even though it is there). Can you point me to some working example so I can rule out I'm doing something wrong, please?

          cimbi Martin Cimbalek added a comment - - edited Hello guys. Thank you for this feature. However, I am not able to get the "managed:" feature work. It simply cannot find the file (even though it is there). Can you point me to some working example so I can rule out I'm doing something wrong, please?
          slide_o_mix Alex Earl added a comment -

          cimbi probably want to ask on the Jenkins Users mailing list. The feature has been in place for a long time (7 years) and has tests. So, asking your question on the mailing list would be best. I no longer maintain this plugin, but someone on the mailing list may be able to help.

          slide_o_mix Alex Earl added a comment - cimbi  probably want to ask on the Jenkins Users mailing list. The feature has been in place for a long time (7 years) and has tests. So, asking your question on the mailing list would be best. I no longer maintain this plugin, but someone on the mailing list may be able to help.

          Will do, thanks!

          cimbi Martin Cimbalek added a comment - Will do, thanks!

          People

            slide_o_mix Alex Earl
            tyrius Jonathan Zimmerman
            Votes:
            2 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: