#66 Fedora CI doesn't show error when PR cannot be merged.
Closed 3 years ago by bgoncalv. Opened 4 years ago by lvrabec.

Fedora CI won't start if PR cannot be merged, but this information is not shown in the PR FLAGS so it looks like CI is not working.

Some error msg in pagure would be more user friendly.

Thanks,
Lukas.


The pipeline starts, but then when it tries to merge the patch it fails and send error message, the problem seems either we are sending the message with wrong format, or pagure failed to process it.

https://jenkins-continuous-infra.apps.ci.centos.org/view/Fedora%20All%20Packages%20Pipeline/job/fedora-rawhide-pr-pipeline/1900/console

The pipeline sent using the new format:

org.centos.prod.ci.dist-git-pr.test.error

{"msg":{"artifact":{"comment_id":29634,"commit_hash":"ac2d797d95ae700332ef8ada99882fd8022bc84a","id":"10","issuer":"lvrabec","repository":"https://src.fedoraproject.org/rpms/udica","type":"pull-request","uid":"a8b615be2ac447c69708903384d82a19"},"contact":{"docs":"https://pagure.io/standard-test-roles","email":"ci@lists.fedoraproject.org","irc":"#fedora-ci","name":"fedora-ci","team":"fedora-ci","url":"https://jenkins-continuous-infra.apps.ci.centos.org/"},"error":{"reason":"Unknown execution error"},"generated_at":"2019-08-27T19:35:01.332Z","pipeline":{"build":"1900","id":"849e8af8-8590-4a8f-a0a3-e00a4373e223","name":"fedora-rawhide-pr-pipeline","stage":{"name":"koji-build"}},"run":{"log":"https://jenkins-continuous-infra.apps.ci.centos.org/job/fedora-rawhide-pr-pipeline/1900/console","rebuild":"https://jenkins-continuous-infra.apps.ci.centos.org/job/fedora-rawhide-pr-pipeline/1900/rebuild/parameterized","url":"https://jenkins-continuous-infra.apps.ci.centos.org/blue/organizations/jenkins/fedora-rawhide-pr-pipeline/detail/fedora-rawhide-pr-pipeline/1900/pipeline/"},"test":{"category":"static-analysis","namespace":"fedora-ci.dist-git-pr","type":"build"},"version":"0.2.1"},"msg_id":"2019-009363f8-357e-4130-a24c-7233256f6198","timestamp":0,"topic":"org.centos.prod.ci.dist-git-pr.test.error"}

https://apps.fedoraproject.org/datagrepper/id?id=2019-009363f8-357e-4130-a24c-7233256f6198&is_raw=true&size=extra-large

and also the old one:

org.centos.prod.ci.pipeline.allpackages-pr.complete

{"msg":{"branch":"master","build_id":"1900","build_url":"https://jenkins-continuous-infra.apps.ci.centos.org/blue/organizations/jenkins/fedora-rawhide-pr-pipeline/detail/fedora-rawhide-pr-pipeline/1900/pipeline/","ci_topic":"org.centos.prod.ci.pipeline.allpackages-pr.complete","comment_id":"29634","commit_hash":"ac2d797d95ae700332ef8ada99882fd8022bc84a","namespace":"rpms","nvr":"","original_spec_nvr":"","ref":"x86_64","repo":"udica","rev":"PR-10","scratch":true,"status":"FAILURE","test_guidance":"''","username":null},"msg_id":"2019-875ff988-2ab0-43f1-b6af-7bbfe66cccf7","timestamp":0,"topic":"org.centos.prod.ci.pipeline.allpackages-pr.complete"}

https://apps.fedoraproject.org/datagrepper/id?id=2019-875ff988-2ab0-43f1-b6af-7bbfe66cccf7&is_raw=true&size=extra-large

I suspect loopabull isn't listening for these messages

Yeah, we will need to patch that ASAP

Will try to prepare a patch for tomorrow

@mvadkert did this ever get fixed up?

Metadata Update from @jimbair:
- Issue assigned to mvadkert

4 years ago

@mvadkert @jimbair if this was just about processing the new message on loopabull, this is done and this ticket can be closed.

We believe this issue has already been fixed.
If for some reason it is not, please reopen it.

Metadata Update from @bgoncalv:
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata