koji

#4051 fixes for list-users
Merged by tkopecek. Opened by mikem.
mikem/koji list-users-updates  into  master

Download 4051.patch

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

~~This is a work in progress. Unit tests still need updating.~~

Also, I should probably separate the db changes here. I needed them for easier debugging, but they aren't needed for the fix

rebased onto aaa836ea9b111b640f238054520565f9cf9422a1

Updated unit tests, dropped the db debug change, added a compat warning, and rebased

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

I only hit a potential issue not-related to the PR itself. What if user inherits permission from disabled group? I would say that expectation is that these permissions don't get inherited at that time. It is probably worth a new issue?

Metadata Update from @relias-redhat:
- Pull-request tagged with: testing-done

Commit 8187761c fixes this pull-request

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