waiverdb

Clone

#326 CI/CD: Fix postmerge job commit flagging
Merged 4 years ago by rayson. Opened 4 years ago by rayson.
rayson/waiverdb cicd-fix-post-merge-flag  into  master

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

  }
 

  def flagCommit(status, percent, comment) {
 

    withPagureCreds {
 

-     it.flagCommit(username: 'c3i-jenkins', uid: 'ci-post-merge', status: status,
 

+     it.flagCommit(username: 'c3i-jenkins', uid: "ci-post-merge-${env.WAIVERDB_GIT_COMMIT.substring(0, 7)}", status: status,
 

        url: env.BUILD_URL, percent: percent, comment: comment, commit: env.WAIVERDB_GIT_COMMIT)
 

    }
 

  }

Pagure doesn't seem to allow to use the same uid for different commits.

Is this also needed in the setBuildStatusOnPagurePR function above?

Oh, never mind, I thought this is for PR status. It makes sense for uid for flagging commits to contain commit ID.

+1

+1

Pull-Request has been merged by rayson
4 years ago
Metadata
None
No Tags
Flags
c3i-jenkins
success (100%)
Build passed.
4 years ago
Changes Summary 1
+1 -1
file changed
openshift/pipelines/templates/waiverdb-build.Jenkinsfile
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.