#4050 updated list-users command fails for 1.34 hub
Closed: Fixed by tkopecek. Opened by mikem.

Recent updates to the list-users command unconditionally pass new options that the current release does not support. This causes the command to fail

$ brew list-users
2024-03-12 13:07:23,460 [ERROR] koji: ParameterError: listUsers() got an unexpected keyword argument 'perm'

There are also some other issues that were missed in review of #3970 and #4008

  • no way to select all usertypes in the cli
  • using --perm overrides any usertype selection with usertype=None
  • there's not real reason to return the permission fields in listUsers since we're always selecting a specific value. It could also be misleading.
  • listUsers only accepts perm arg as a string, which is inconsistent with other calls in the api that accept such an arg
  • duplicate user_perms.active IS TRUE clause

Metadata Update from @mikem:
- Custom field Size adjusted to None

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.34.1

Metadata Update from @tkopecek:
- Issue tagged with: testing-ready

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.35 (was: 1.34.1)

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

Commit 8187761c fixes this issue

Commit 32239f2c fixes this issue

This issue has been migrated to Fedora Forge:
https://forge.fedoraproject.org/koji/koji/issues/4050

Please continue any further discussion there.

Metadata
Related Pull Requests