#3154 CICO Pipeline: Let the script checkout the PR branch
Merged 6 years ago by pingou. Opened 6 years ago by bstinson.
bstinson/pagure dont-fight-the-ci-script  into  master

file modified
+3 -6
@@ -27,6 +27,8 @@ 

              ).trim().tokenize(' ')

          env.DUFFY_NODE=duffy_rtn[0]

          env.SSID=duffy_rtn[1]

+         env.BRANCH=params.BRANCH

+         env.REPO=params.REPO

      }

  

      try {
@@ -36,12 +38,7 @@ 

          }

  

          stage('Clone Test Suite') {

-             if (params.BRANCH){

-                 println "Checking out branch: ${params.BRANCH}"

-                 onmyduffynode "git clone -b \"${params.BRANCH}\" --single-branch --depth 1 https://pagure.io/pagure.git"

-             } else {

-                 onmyduffynode "git clone --single-branch --depth 1 https://pagure.io/pagure.git"

-             }

+             onmyduffynode "git clone --single-branch --depth 1 https://pagure.io/pagure.git"

          }

  

          stage('Run Test Suite') {

We were trying to have jenkins checkout the proper PR branch but the ci script already does this.

The CI script just needs a copy of itself from master, and it will do the right thing.

1 new commit added

  • add the REPO to the environment as well
6 years ago

rebased onto 5362ad6

6 years ago

Commit 44116ae fixes this pull-request

Pull-Request has been merged by pingou

6 years ago