In Foreman core, admins can bypass required checks on PRs and merge even if one of them is red. This is very useful e.g. when a PR fails on some known transient issue or due to network errors.
In Katello, red checks can’t be bypassed, meaning that CI has to be retriggered until all checks are green, sometimes multiple times (for example: https://github.com/Katello/katello/pull/8676). Since tests commonly take over an hour, this can cause delays to merged known good PRs, as well as waste resources for unneeded CI runs.
I suggest changing the katello repo settings to allow admins to bypass the required checks, similar to the Foreman repo, or not mark the tests as a required, and trust all maintainer to know what they are doing when merging a PR that failed tests.
What do others think, specifically @katello team?
- Keep it as is
- Allow admins to bypass required checks
- Unmark tests as a