Skip to content

ack: support a required bot approval

Jarod Wilson requested to merge jwilsonrh/kernel-webhooks:botack into main

Support requiring a bot approval on MRs. This requires adding a new "Ready For Merge" rule to all project default approval rules, with one required approval from cki-kwf-bot, and bumping the number of overall required approvals on an MR from two to three, but we'll also be accounting for the bot approval not yet being present to still get to Acks::OK, and at the points where we would normally add or remove the readyForMerge label, we'll also have cki-kwf-bot approve or unapprove the MR. This prevents the merge button from even showing up on a merge request without readyForMerge on it.

Fixes: #282 (closed)

[skip coverage check]

Signed-off-by: Jarod Wilson jarod@redhat.com

Edited by Jarod Wilson

Merge request reports