On 24 May 2013 10:57, Kevin Fenzi <kevin@scrye.com> wrote:
On Fri, 24 May 2013 11:44:53 -0500
"Jason L Tibbitts III" <tibbs@math.uh.edu> wrote:

> Please forgive me if this isn't the right format for this request;
> it's been a long time since I've done one of these.
>
> I need to push the following change to the review-stats code through
> puppet:
>
> diff --git a/modules/review-stats/files/review-stats.py
> b/modules/review-stats/files/review-stats.py index 2c59752..e494168
> 100755 --- a/modules/review-stats/files/review-stats.py
> +++ b/modules/review-stats/files/review-stats.py
> @@ -205,7 +205,7 @@ def run_query(bz):
>                  bugdata[bug.id]['hidden'] = 1
>
>      # Get the status of each "interesting" bug
> -    for i in seq_max_split(alldeps, 500):
> +    for i in seq_max_split(alldeps, 100):
>          for bug in filter(None, bz._proxy.Bug.get_bugs({'ids':i,
> 'permissive': 1})['bugs']): interesting[bug['id']] = bug
>
> RHBugzilla is pretty broken right now, and I'm working on fixing this
> code upstream in a better way which I will push once we unfreeze, but
> this is the minimal fix that appears to at least get the script to
> complete without throwing proxy errors.  It takes ages to run but it
> does eventually finish.

Seems like a simple change and it's not working at all now, so +1 here.

I would say drop it down to 50 for the moment. The 4.4 upgrade is undergoing a lot of changes to try and get it working but it could be a while (and it could get worse before it gets better).


--
Stephen J Smoogen.