koji

#2098 move admin force usage to assert_policy
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue1930a  into  master

Download 2098.patch

Fixes: https://pagure.io/koji/issue/1930

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

:thumbsup:

I notice we're losing a some details from the original log messages (e.g. tag name, package name). I suppose that may be a price we just have to pay to systematize this.

It might be valuable to log the reason for the overridden policy denial in the check_policy message.

Overall looks good

1 new commit added

  • add refusal reason to log

Commit 7e169c81 fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done
Metadata
None

testing-done testing-ready

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.