fedora-ci / monitor-gating

#10 Test for open issues, and abort if present.
Merged by pingou. Opened by asaleh.
fedora-ci/ asaleh/monitor-gating check_for_open_issues  into  master

Download 10.patch

Currently just Pagure issues, because that is the one issue that is blocking us :)

Hm, this ties the code to a very particular issue, once that issue is fixed there could be a different one.
It also only affects single build while the issue with waiverdb will affect the entire pipeline (ie: both)

rebased onto 4c5a74896810e50f28c723f21ee0ee66d5dd4bb6

Not black/pep8 compliant :)

Not black/pep8 compliant :)

Left over debugging?

I guess we should pull the environment from the configuration file as well?

Do we want to handle exceptions here?
Do we want doc strings?

1 new commit added

  • Flake fixes, added pre-commit and flake config.

I added a simple doc-string to the utility function.

I decided to just log on exception, and assume there are no blockers.

So far we only run runner in staging, so hard-coding it for now should be fine.

The function will print all of the issues it found, so I want to run the query at the start of the runner, even though we don't use the result.

1 new commit added

  • Tags to search blokcing issues no longer hardcoded.

Nice, I like this.

One note, should we add a blob in the README for the precommit?

I'll merge this as is, we can improve the README in another PR :)

Pull-Request has been merged by pingou
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.