#3157 quote the environment variables when passing on to the duffy node
Merged 6 years ago by pingou. Opened 6 years ago by bstinson.
bstinson/pagure ci-quote-fix  into  master

file modified
+1 -1
@@ -1,7 +1,7 @@ 

  def onmyduffynode(script){

      ansiColor('xterm'){

          timestamps{

-             sh 'ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -l root ${DUFFY_NODE}.ci.centos.org -t REPO=${REPO} BRANCH=${BRANCH} "' + script + '"'

+             sh 'ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -l root ${DUFFY_NODE}.ci.centos.org -t \"REPO=${REPO} BRANCH=${BRANCH}\" "' + script + '"'

          }

      }

  }

This should pass these correctly into the script now.

Commit 7ee868c fixes this pull-request

Pull-Request has been merged by pingou

6 years ago
Metadata