koji

#3071 remove koji.listFaults() method
Closed: Fixed by julian8628. Opened by ktdreyer.

Closed: Fixed
Reopen Issue

Nothing uses koji.listFaults(). I searched all of GitHub and found no references to 3rd-party code either.

Let's remove this method from the tree.

Looking through the git history, it appeared in the initial code drop, and nothing has ever called this method.

Metadata Update from @tkopecek:
- Custom field Size adjusted to None
- Issue set to the milestone: 1.27
- Issue tagged with: deprecation

@mikem any concerns with directly dropping this? (or rather deprecation warning?)

Well, anything in the main lib needs to have a deprecation period.

My only concern is that we might want something like this functionality (but probably not this implementation) in the future. It's currently tricky to add new faults because an older client will not know about them. It might be nice if they were discoverable.

PR #3082

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

Metadata Update from @jobrauer:
- Issue tagged with: testing-done

Commit 537439c6 fixes this issue

Commit 9e6be3f9 relates to this ticket

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

Please continue any further discussion there.
Metadata
None

deprecation testing-done testing-ready

None
None
Low
None
Related Pull Requests
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.