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

Add an option to override destination folder on master node

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • copy-to-slave-plugin
    • None
    • Jenkins ver. 1.424

      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.

          [JENKINS-10691] Add an option to override destination folder on master node

          Thomas Fields added a comment -

          Hi Romain,

          Did you find the time to add this feature?

          Regards,
          Tom.

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

          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

          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

          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.

          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/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/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

          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.

          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.

          Thomas Fields added a comment -

          Attached log file.

          Thomas Fields added a comment - Attached log file.

          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.

          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.

          Romain Seguy added a comment -

          I've had no time for that.

          Romain Seguy added a comment - I've had no time for that.

          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.

          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.

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

              Created:
              Updated:
              Resolved: