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

groovy script can not run on node specified for the job

      script can not run on node specified for the job
      when trying to use the groovy script option for active choice plugin, the script run only on master. need to add ability to distinguish the machine on which the job will run and run the groovy script on that machine.

      point to consider:
      jobs can run on several machines so adding just a checkbox to run script on remote like the implementation on dynamic paramter plugin is probably not enough. instead it would be better to specify the node tag on which you want to run the script.

          [JENKINS-29969] groovy script can not run on node specified for the job

          Bikas Anand added a comment -

          Thanks Ioannis!
          But it can not solve my issue as it may need to run an http server on the slave to fetch the file this way.

          In my believe, the feature the present issue talks about, is like a "code copy paste" easy to fix as this feature is already present in DynamicParameter plugin.
          If not possible, let me flip back to another option (Seeking ReactiveParameter feature in DynamicParameter plugin) and see if it can help. Just created a request for this.
          https://issues.jenkins-ci.org/browse/JENKINS-35495

          Bikas Anand added a comment - Thanks Ioannis! But it can not solve my issue as it may need to run an http server on the slave to fetch the file this way. In my believe, the feature the present issue talks about, is like a "code copy paste" easy to fix as this feature is already present in DynamicParameter plugin. If not possible, let me flip back to another option (Seeking ReactiveParameter feature in DynamicParameter plugin) and see if it can help. Just created a request for this. https://issues.jenkins-ci.org/browse/JENKINS-35495

          Hi Bikas,

          From JENKINS-35495 "It seems ActiveChoiceParamer team is reluctant to provide this feature. ".

          Not really. Just not enough time. We try to squeeze as many issues as possible in each release, but sometimes issues are prioritised, and other issues take a longer time to be implemented.

          This is easier to implement in active-choices, then to port the reactive parameter type to dynamic-parameter (the reactive parameter was created with JavaScript, and by some Java code as well).

          That said, once we start another development cycle for BioUno, will start triaging issues in our plug-ins and tools, and try our best at releasing new versions.

          Should you have a pull request (to either active-choices or dynamic-parameter), feel free to submit it. Most pull requests are merged when we cut a new release as long as we do not break backward compatibility.

          Sorry for the delay to fix this issue everyone.
          Cheers,

          Bruno

          Bruno P. Kinoshita added a comment - Hi Bikas, From JENKINS-35495 "It seems ActiveChoiceParamer team is reluctant to provide this feature. ". Not really. Just not enough time. We try to squeeze as many issues as possible in each release, but sometimes issues are prioritised, and other issues take a longer time to be implemented. This is easier to implement in active-choices, then to port the reactive parameter type to dynamic-parameter (the reactive parameter was created with JavaScript, and by some Java code as well). That said, once we start another development cycle for BioUno, will start triaging issues in our plug-ins and tools, and try our best at releasing new versions. Should you have a pull request (to either active-choices or dynamic-parameter), feel free to submit it. Most pull requests are merged when we cut a new release as long as we do not break backward compatibility. Sorry for the delay to fix this issue everyone. Cheers, Bruno

          Dakota Brown added a comment -

          It'd be super nice to be able to provide a label if the "Remote Script" checkbox isn't an option

          Dakota Brown added a comment - It'd be super nice to be able to provide a label if the "Remote Script" checkbox isn't an option

          Hi Dakota, promising that will try to include this feature in the next release. Or at least make progress on the work for it. Winter is coming here in Oceania, so about to start a new cycle for active-choices.

          Bruno P. Kinoshita added a comment - Hi Dakota, promising that will try to include this feature in the next release. Or at least make progress on the work for it. Winter is coming here in Oceania, so about to start a new cycle for active-choices.

          Dakota Brown added a comment -

          This is going to help a bunch.

           

          I have an AWS Jenkins slave that I'd love to use to grab a list of AWS-related parameter choices via AWSCLI while also depending on another parameter like AWS_REGION as input.

          I work for a large company moving to AWS and I know this would be hugely useful in our migration efforts

          Dakota Brown added a comment - This is going to help a bunch.   I have an AWS Jenkins slave that I'd love to use to grab a list of AWS-related parameter choices via AWSCLI while also depending on another parameter like AWS_REGION as input. I work for a large company moving to AWS and I know this would be hugely useful in our migration efforts

          Bipul Raj added a comment -

          Is fix for this coming anytime soon? This is really badly needed.

          Bipul Raj added a comment - Is fix for this coming anytime soon? This is really badly needed.

          Not yet, there is an alpha release, waiting further tests. Once that's OK, will merge the branch and continue triaging issues OR release as-is and start a new testing & bug fix.

          However, should someone submits a pull request, I can have a look on a weekend and probably include in the release if the code looks OK.

          Bruno P. Kinoshita added a comment - Not yet, there is an alpha release, waiting further tests. Once that's OK, will merge the branch and continue triaging issues OR release as-is and start a new testing & bug fix. However, should someone submits a pull request, I can have a look on a weekend and probably include in the release if the code looks OK.

          Bipul Raj added a comment -

          kinow Can you please give a summary what changes would be needed to add this support? If doable for me, I can try it out and test it.

          Bipul Raj added a comment - kinow Can you please give a summary what changes would be needed to add this support? If doable for me, I can try it out and test it.

          Diego added a comment -

          Can we expect a fix for this bug?

          I need to get a list of directories on the slave node for Active Choices plugin, but this is not possible from the master node 

          Diego added a comment - Can we expect a fix for this bug? I need to get a list of directories on the slave node for Active Choices plugin, but this is not possible from the master node 

          Looking also for this change, it will make Active Choice more scalable 

          Selma Ben Said added a comment - Looking also for this change, it will make Active Choice more scalable 

            kinow Bruno P. Kinoshita
            doronz doron zukerman
            Votes:
            6 Vote for this issue
            Watchers:
            15 Start watching this issue

              Created:
              Updated: