fedora-zuul-jobs-config

#207 Cancel still running scratch build
Merged by zuul. Opened by lecris.
lecris/fedora-zuul-jobs-config cancel-scratch-build  into  master

Download 207.patch

Requires: https://pagure.io/zuul-distro-jobs/pull-request/159

Fedora-CI counterpart: https://github.com/fedora-ci/dist-git-build-pipeline/pull/42

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/9aeb8ee9a6fe4825938dbb5aa2b8b592

rebased onto 564e5647296f98ddde5ff739b1f7512d082e6d0d

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/bf369ccd8c084eb3ae468dea5d803a4d

rebased onto 564e5647296f98ddde5ff739b1f7512d082e6d0d

rebased onto 564e5647296f98ddde5ff739b1f7512d082e6d0d

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/bb1a08cd7aaa44a9a4bc33df95fd8250

LGTM

Here task_id won't be available, run, post-run are different Ansible playbook run context.

Sorry I did the review to fast

That was my thought as well, but corvus suggested that it should work if we use set_fact. The discussion for reference: https://matrix.to/#/!yuuvjJSOEGSfTzxOjK:opendev.org/$1n2a7j1Hu_N0f7XlPsWt6GfMfP1yXjUMMJUvlVQReJg?via=matrix.org&via=opendev.org&via=reszka.org

Does feel like some dark magic going on there, and we are not the only ones surprised by that in that chat
Edited by lecris

Will go ahead with merging these and will test it over in https://src.fedoraproject.org/rpms/python-scikit-build-core/pull-request/73

Metadata Update from @lecris:
- Pull-request tagged with: gateit

Build succeeded (gate pipeline).
https://fedora.softwarefactory-project.io/zuul/buildset/c0437d34f4484b46acf03ac2d18103e9

Pull-Request has been merged by zuul

Well, the intended logic did not go as planned, but at least it did not make things worse.

2025-09-03 11:15:24.809439 | RUN END RESULT_ABORTED: [trusted : pagure.io/fedora-zuul-jobs-config/playbooks/koji/build-ng.yaml@master]
2025-09-03 11:15:24.811196 | POST-RUN START: [trusted : pagure.io/fedora-zuul-jobs-config/playbooks/koji/cleanup.yaml@master]
2025-09-03 11:15:25.386010 | 
2025-09-03 11:15:25.386137 | PLAY [all]
2025-09-03 11:15:25.405589 | 
2025-09-03 11:15:25.405698 | TASK [include_role : fedpkg-build-cleanup]
2025-09-03 11:15:25.425520 | container | ok
2025-09-03 11:15:25.438099 | 
2025-09-03 11:15:25.438178 | TASK [fedpkg-build-cleanup : Get the current state of Koji task]
2025-09-03 11:15:25.728990 | container | skipping: Conditional result was False
2025-09-03 11:15:25.741314 | 
2025-09-03 11:15:25.741466 | TASK [fedpkg-build-cleanup : Get koji task state]
2025-09-03 11:15:25.789924 | container | skipping: Conditional result was False
2025-09-03 11:15:25.808917 | 
2025-09-03 11:15:25.809075 | TASK [fedpkg-build-cleanup : Cancel koji scratch build]
2025-09-03 11:15:26.277035 | container | skipping: Conditional result was False
2025-09-03 11:15:26.317889 | 
2025-09-03 11:15:26.317975 | PLAY RECAP
2025-09-03 11:15:26.318034 | container | ok: 0 changed: 0 unreachable: 0 failed: 0 skipped: 3 rescued: 0 ignored: 0
2025-09-03 11:15:26.318070 | 
2025-09-03 11:15:26.411615 | POST-RUN END RESULT_ABORTED: [trusted : pagure.io/fedora-zuul-jobs-config/playbooks/koji/cleanup.yaml@master]

I'll add some debug to check corvus's claim and get back to them when matrix is back online

Found the issue, just waiting on matrix to recover and I'll ask corvus for some guidance

2025-09-03 11:43:50.745837 | TASK [fedpkg-build-cleanup : debug]
2025-09-03 11:43:50.815201 | task_id = not defined
Edited by lecris
Metadata
None

gateit

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.