waiverdb

#261 CI/CD: Make a comment on pull-request build failure
Merged by lholecek. Opened by rayson.
rayson/waiverdb cicd-comment-on-pr  into  master

Download 261.patch

Following up #256. Leave a message when a CI build of pull-request
fails.

Examples: https://pagure.io/fork/rayson/waiverdb/pull-request/9

Bot accounts might be needed to make distinguish between comments made by human or program (https://pagure.io/waiverdb/issue/258).

:+1:

Does handle errors (e.g. when HTTP status is not 201)?
Edited by lholecek

@lholecek According to https://github.com/jenkinsci/http-request-plugin/blob/95e4dde1593508e6dcad997b335f341965ab987b/src/main/java/jenkins/plugins/http_request/HttpRequest.java#L394 , by default the Jenkins HTTP Request plugin treats HTTP code from 100-399 as a success. Otherwise an exception will be thrown. BTW, Pagure API is not restful. It always returns 200 on success.

+1

Commit 1f43f3a9 fixes this pull-request

Pull-Request has been merged by lholecek

Pull-Request has been merged by lholecek
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.