On 15/09/21 17:38, Till Maas wrote: > On Mon, Sep 13, 2021 at 12:49:21PM -0600, Tim Flink wrote: > >> + # this is a hotfix hack to work around the sudden config change in rhbz where the max >> + # number of bugs returned for a query is 20 >> + # it seems to be working for now but may need more work going forward >> + while last_query_len == BUGZILLA_QUERY_LIMIT: >> + >> + new_query = self.get_bz_query(tracker, last_update, offset=len(buglist)) >> + new_buglist = self.bz.query(new_query) >> + buglist.extend(new_buglist) >> + last_query_len = len(new_buglist) > FYI: This algorithm has a race condition, see also: > https://github.com/python-bugzilla/python-bugzilla/issues/149#issuecomment-918213094 > > Cheers > Till From the API docs: > If you set the limit equal to zero, then all matching results will be > returned instead I suppose setting limit=0 would be enough. I made some local tests with review-stats code (which is also affected by bz change) and that seems to work. However, review-stats uses a logged-in user for the query, I don't know if that's the case for blockerbugs also. Mattia _______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
