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

Throw appropriate error when the code is not in node block

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The pipeline example for sshCommand provided in the Github page is wrong, it gives a NullPointException in my installation:

      java.lang.NullPointerException
      	at org.jenkinsci.plugins.sshsteps.steps.CommandStep$Execution.run(CommandStep.java:71)
      	at org.jenkinsci.plugins.sshsteps.util.SSHStepExecution$1$1.call(SSHStepExecution.java:77)
      	at hudson.security.ACL.impersonate(ACL.java:290)
      	at org.jenkinsci.plugins.sshsteps.util.SSHStepExecution$1.run(SSHStepExecution.java:72)
      	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
      	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
      	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
      	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
      	at java.lang.Thread.run(Thread.java:748)
      Finished: FAILURE

      However, after adding pipelines, stages and steps wrappers around it, it worked fine:

      def remote = [:]
      remote.name = 'test'
      remote.host = 'test.domain.com'
      remote.user = 'root'
      remote.password = 'password'
      remote.allowAnyHosts = true
      pipeline { 
          agent any 
          stages {
              stage('SSH') {
                  steps{
                      sshCommand remote: remote, command: "ls -l /"
                  }
              }
          }
      }
      

        Attachments

          Activity

          Hide
          nrayapati Naresh Rayapati added a comment -

          Cassio Binkowski That was a scripted pipeline example, needs to be called in node as in example at [1]. Thanks for reporting, I will update all of these examples while working on JENKINS-52226

          node {
            def remote = [:]
            remote.name = 'test'
            remote.host = 'test.domain.com'
            remote.user = 'root'
            remote.password = 'password'
            remote.allowAnyHosts = true
            stage('Remote SSH') {
              sshCommand remote: remote, command: "ls -lrt"
              sshCommand remote: remote, command: "for i in {1..5}; do echo -n \"Loop \$i \"; date ; sleep 1; done"
            }
          }
          

          [1] https://github.com/jenkinsci/ssh-steps-plugin#withcredentials

          Show
          nrayapati Naresh Rayapati added a comment - Cassio Binkowski That was a scripted pipeline example, needs to be called in node as in example at [1] . Thanks for reporting, I will update all of these examples while working on JENKINS-52226 node { def remote = [:] remote.name = 'test' remote.host = 'test.domain.com' remote.user = 'root' remote.password = 'password' remote.allowAnyHosts = true stage( 'Remote SSH' ) { sshCommand remote: remote, command: "ls -lrt" sshCommand remote: remote, command: " for i in {1..5}; do echo -n \" Loop \$i \ "; date ; sleep 1; done" } } [1] https://github.com/jenkinsci/ssh-steps-plugin#withcredentials
          Hide
          nrayapati Naresh Rayapati added a comment -

          Made a pull request to throw appropriate error instead of throwing  java.lang.NullPointerException

          https://github.com/jenkinsci/ssh-steps-plugin/pull/3

          CC: Gabriel Henkes Wuchen Wang

          Show
          nrayapati Naresh Rayapati added a comment - Made a pull request to throw appropriate error instead of throwing  java.lang.NullPointerException https://github.com/jenkinsci/ssh-steps-plugin/pull/3 CC: Gabriel Henkes Wuchen Wang
          Hide
          nrayapati Naresh Rayapati added a comment -

          Merged above pull request, will release it along with 1.0.1

          Thanks again for reporting! Cassio Binkowski

          Show
          nrayapati Naresh Rayapati added a comment - Merged above pull request, will release it along with 1.0.1 Thanks again for reporting! Cassio Binkowski
          Hide
          nrayapati Naresh Rayapati added a comment -

          Released this with 1.1.0. Closing.. Thank you.

          Show
          nrayapati Naresh Rayapati added a comment - Released this with 1.1.0 . Closing.. Thank you.

            People

            Assignee:
            nrayapati Naresh Rayapati
            Reporter:
            cassioiks Cassio Binkowski
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: