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

Add an option to override destination folder on master node

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • copy-to-slave-plugin
    • None
    • Jenkins ver. 1.424

    Description

      Hi there,

      I'd like to see a new option added to the Copy to slave plugin.

      At the moment, if sub-jobs from a matrix project are built on slave nodes, the plugin provides the option to "Copy files back to the job's workspace on the master node". Is it possible to add an option to override the destination path on the master node? Copying back to the job's workspace is not always ideal - in my case I'd like to copy to ${JENKINS_HOME}/current_build - can this feature be added?

      Regards,
      Tom.

      Attachments

        Activity

          rseguy Romain Seguy added a comment -

          That should be possible. I'll take a look when I'm back from my vacations.

          rseguy Romain Seguy added a comment - That should be possible. I'll take a look when I'm back from my vacations.
          tfields Thomas Fields added a comment -

          Hi Romain,

          Did you find the time to add this feature?

          Regards,
          Tom.

          tfields Thomas Fields added a comment - Hi Romain, Did you find the time to add this feature? Regards, Tom.
          rseguy Romain Seguy added a comment -

          Hi Tom,

          I've had no time to work on that. If you feel you can implement it, just don't hesitate and attach a patch to this ER.

          Thx,
          Romain

          rseguy Romain Seguy added a comment - Hi Tom, I've had no time to work on that. If you feel you can implement it, just don't hesitate and attach a patch to this ER. Thx, Romain
          rseguy Romain Seguy added a comment - - edited

          Thomas,

          Please find attached a build (copy-to-slave.hpi) with the new feature. Please confirm it works fine. If so, I'll roll out a new release.

          rseguy Romain Seguy added a comment - - edited Thomas, Please find attached a build ( copy-to-slave.hpi ) with the new feature. Please confirm it works fine. If so, I'll roll out a new release.

          Code changed in jenkins
          User: rseguy
          Path:
          README.md
          pom.xml
          src/main/java/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier.java
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.jelly
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.properties
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config_fr.properties
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/help-destinationFolder.html
          http://jenkins-ci.org/commit/copy-to-slave-plugin/62d8cfd9121c0f32e2d9aa381b84588bc5bd6152
          Log:
          [FIXED JENKINS-10691] Added an option to override destination folder on master node

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: rseguy Path: README.md pom.xml src/main/java/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier.java src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.jelly src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.properties src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config_fr.properties src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/help-destinationFolder.html http://jenkins-ci.org/commit/copy-to-slave-plugin/62d8cfd9121c0f32e2d9aa381b84588bc5bd6152 Log: [FIXED JENKINS-10691] Added an option to override destination folder on master node

          Code changed in jenkins
          User: rseguy
          Path:
          README.md
          pom.xml
          src/main/java/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier.java
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.jelly
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.properties
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config_fr.properties
          src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/help-destinationFolder.html
          http://jenkins-ci.org/commit/copy-to-slave-plugin/62d8cfd9121c0f32e2d9aa381b84588bc5bd6152
          Log:
          [FIXED JENKINS-10691] Added an option to override destination folder on master node

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: rseguy Path: README.md pom.xml src/main/java/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier.java src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.jelly src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config.properties src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/config_fr.properties src/main/resources/com/michelin/cio/hudson/plugins/copytoslave/CopyToMasterNotifier/help-destinationFolder.html http://jenkins-ci.org/commit/copy-to-slave-plugin/62d8cfd9121c0f32e2d9aa381b84588bc5bd6152 Log: [FIXED JENKINS-10691] Added an option to override destination folder on master node
          tfields Thomas Fields added a comment -

          Hi Romain,

          Thank you for updating the plugin.

          I clicked "Override destination folder" and entered a new path on the master node and the files were copied back as expected. However, when I checked the log I got the output attached.

          Regards,
          Tom.

          tfields Thomas Fields added a comment - Hi Romain, Thank you for updating the plugin. I clicked "Override destination folder" and entered a new path on the master node and the files were copied back as expected. However, when I checked the log I got the output attached. Regards, Tom.
          tfields Thomas Fields added a comment -

          Attached log file.

          tfields Thomas Fields added a comment - Attached log file.
          tfields Thomas Fields added a comment -

          Hi Romain,

          Did you get chance to look at the log file? Any ideas what's going wrong?

          Cheers,
          Tom.

          tfields Thomas Fields added a comment - Hi Romain, Did you get chance to look at the log file? Any ideas what's going wrong? Cheers, Tom.
          rseguy Romain Seguy added a comment -

          I've had no time for that.

          rseguy Romain Seguy added a comment - I've had no time for that.
          tfields Thomas Fields added a comment -

          Hi Romain,

          I've just tried your updated plugin again and this time I didn't get any errors. Obviously lots of stuff has changed on my end so I'll keep you posted if it comes back. Can the update plugin be released into the wild?

          Regards,
          Tom.

          tfields Thomas Fields added a comment - Hi Romain, I've just tried your updated plugin again and this time I didn't get any errors. Obviously lots of stuff has changed on my end so I'll keep you posted if it comes back. Can the update plugin be released into the wild? Regards, Tom.

          People

            rseguy Romain Seguy
            tfields Thomas Fields
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: