Freeze break request: Update python-bugzilla on fas*, bapp01, app*

[Kevin Fenzi]

After the bugzilla upgrade this weekend a few things we run related to
bugzilla are having issues: 

1) on fas01, the script that sets fedorabugs and fedora_contrib on users
in those fedora groups isn't working. 

2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
working. 

3) pkgdb, which runs on app servers isnt' able to handle adding new
packages. (we aren't sure what the error here is, but is likely bz
related). 

I have taken a snapshot of python-bugzilla and built a package of it. 
Changes: 
http://git.fedorahosted.org/git/?p=python-bugzilla.git
Scratch build: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=4091680
(This is just a git head checkout build)

1) I have tested on fas01.stg the script that sets permissions
against partner-bugzilla.redhat.com and the updated python-bugzilla. It
no longer errors and completes normally. 

2) gather easyfix needs more changes and is not fixed by
python-bugzilla update. Look for fix after this email from pingou. 

3) We aren't sure the new python-bugzilla will fix the pkgdb adding
packages issue, but should get us closer to a fix. 

I would like to add the new python-bugzilla to our infrastructure repo
(for now until there's an official one in epel) and update fas*,
bapp01, and app*. In the event of failure, we can downgrade back to the
previous version with a yum downgrade. 

+1s?

kevin

[Seth Vidal]

On Mon, 21 May 2012 10:38:14 -0600
Kevin Fenzi <kevin(a)scrye.com&gt; wrote:

>  After the bugzilla upgrade this weekend a few things we run related
> to
>  bugzilla are having issues: 
>  
>  1) on fas01, the script that sets fedorabugs and fedora_contrib on
>  users in those fedora groups isn't working. 
>  
>  2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
>  working. 
>  
>  3) pkgdb, which runs on app servers isnt' able to handle adding new
>  packages. (we aren't sure what the error here is, but is likely bz
>  related). 
>  
>  I have taken a snapshot of python-bugzilla and built a package of it. 
>  Changes: 
>  http://git.fedorahosted.org/git/?p=python-bugzilla.git
>  Scratch build: 
>  http://koji.fedoraproject.org/koji/taskinfo?taskID=4091680
>  (This is just a git head checkout build)
>  
>  1) I have tested on fas01.stg the script that sets permissions
>  against partner-bugzilla.redhat.com and the updated python-bugzilla.
>  It no longer errors and completes normally. 
>  
>  2) gather easyfix needs more changes and is not fixed by
>  python-bugzilla update. Look for fix after this email from pingou. 
>  
>  3) We aren't sure the new python-bugzilla will fix the pkgdb adding
>  packages issue, but should get us closer to a fix. 
>  
>  I would like to add the new python-bugzilla to our infrastructure repo
>  (for now until there's an official one in epel) and update fas*,
>  bapp01, and app*. In the event of failure, we can downgrade back to
>  the previous version with a yum downgrade. 
>  
>  +1s?
>  
>  kevin 
+1 - I don't see how we have much choice but to go forward here.

-sv

[Toshio Kuratomi]

+1 from me

On Mon, May 21, 2012 at 9:38 AM, Kevin Fenzi <kevin(a)scrye.com&gt; wrote:
>  After the bugzilla upgrade this weekend a few things we run related
> to
>  bugzilla are having issues:
>
>  1) on fas01, the script that sets fedorabugs and fedora_contrib on users
>  in those fedora groups isn't working.
>
>  2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
>  working.
>
>  3) pkgdb, which runs on app servers isnt' able to handle adding new
>  packages. (we aren't sure what the error here is, but is likely bz
>  related).
>
>  I have taken a snapshot of python-bugzilla and built a package of it.
>  Changes:
>  http://git.fedorahosted.org/git/?p=python-bugzilla.git
>  Scratch build:
>  http://koji.fedoraproject.org/koji/taskinfo?taskID=4091680
>  (This is just a git head checkout build)
>
>  1) I have tested on fas01.stg the script that sets permissions
>  against partner-bugzilla.redhat.com and the updated python-bugzilla. It
>  no longer errors and completes normally.
>
>  2) gather easyfix needs more changes and is not fixed by
>  python-bugzilla update. Look for fix after this email from pingou.
>
>  3) We aren't sure the new python-bugzilla will fix the pkgdb adding
>  packages issue, but should get us closer to a fix.
>
>  I would like to add the new python-bugzilla to our infrastructure repo
>  (for now until there's an official one in epel) and update fas*,
>  bapp01, and app*. In the event of failure, we can downgrade back to the
>  previous version with a yum downgrade.
>
>  +1s?
>
>  kevin
>
>  _______________________________________________
>  infrastructure mailing list
>  infrastructure(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Pierre-Yves Chibon]

On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  2) gather easyfix needs more changes and is not fixed by
>  python-bugzilla update. Look for fix after this email from pingou.  
The said changes are in the following commit:
http://fpaste.org/7AHP/
Can I have 2 +1 to push this commit in puppet?

Thanks,
Pierre

[Toshio Kuratomi]

+1

On Mon, May 21, 2012 at 10:14 AM, Pierre-Yves Chibon
<pingou(a)pingoured.fr&gt; wrote:
>  On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
> > 2) gather easyfix needs more changes and is not fixed by
> > python-bugzilla update. Look for fix after this email from pingou.
>
>  The said changes are in the following commit:
>  http://fpaste.org/7AHP/
>  Can I have 2 +1 to push this commit in puppet?
>
>  Thanks,
>  Pierre
>  _______________________________________________
>  infrastructure mailing list
>  infrastructure(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Kevin Fenzi]

On Mon, 21 May 2012 19:14:09 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:

>  On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > 2) gather easyfix needs more changes and is not fixed by
>  > python-bugzilla update. Look for fix after this email from pingou. 
>  
>  The said changes are in the following commit:
>  http://fpaste.org/7AHP/
>  Can I have 2 +1 to push this commit in puppet? 
+1 here, but wait for my python-bugzilla update to finish if you can. 

kevin

[Pierre-Yves Chibon]

On Mon, 2012-05-21 at 19:14 +0200, Pierre-Yves Chibon wrote:
>  On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > 2) gather easyfix needs more changes and is not fixed by
>  > python-bugzilla update. Look for fix after this email from pingou. 
>  
>  The said changes are in the following commit:
>  http://fpaste.org/7AHP/
>  Can I have 2 +1 to push this commit in puppet? 
hm apparently one was not enough, so there is a second patch to make the
logging module quiet:
http://fpaste.org/5Q4X/

Thanks,
Pierre

[Kevin Fenzi]

On Mon, 21 May 2012 20:38:42 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:

>  On Mon, 2012-05-21 at 19:14 +0200, Pierre-Yves Chibon wrote:
>  > On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > > 2) gather easyfix needs more changes and is not fixed by
>  > > python-bugzilla update. Look for fix after this email from
>  > > pingou. 
>  > 
>  > The said changes are in the following commit:
>  > http://fpaste.org/7AHP/
>  > Can I have 2 +1 to push this commit in puppet?
>  
>  hm apparently one was not enough, so there is a second patch to make
>  the logging module quiet:
>  http://fpaste.org/5Q4X/ 
+1, would like output from the script when it fails. 

kevin

[Seth Vidal]

On Mon, 21 May 2012 20:38:42 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:

>  On Mon, 2012-05-21 at 19:14 +0200, Pierre-Yves Chibon wrote:
>  > On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > > 2) gather easyfix needs more changes and is not fixed by
>  > > python-bugzilla update. Look for fix after this email from
>  > > pingou. 
>  > 
>  > The said changes are in the following commit:
>  > http://fpaste.org/7AHP/
>  > Can I have 2 +1 to push this commit in puppet?
>  
>  hm apparently one was not enough, so there is a second patch to make
>  the logging module quiet:
>  http://fpaste.org/5Q4X/
>   
+1 - for the record I have much distaste for the logging module :(

-sv

[Pierre-Yves Chibon]

On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  After the bugzilla upgrade this weekend a few things we run related
> to
>  bugzilla are having issues: 
>  
>  1) on fas01, the script that sets fedorabugs and fedora_contrib on
>  users in those fedora groups isn't working. 
>  
>  2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
>  working. 
>  
>  3) pkgdb, which runs on app servers isnt' able to handle adding new
>  packages. (we aren't sure what the error here is, but is likely bz
>  related).  
4) the review-stats script is having problems.
I spent some time on this and I have a patch that runs fine on my
machine against the official bugzilla and with the 'new' release of
python-bugzilla.
It is a little more heavy than the previous patches:
http://fpaste.org/dwwg/

Pierre

[Pierre-Yves Chibon]

On Tue, 2012-05-22 at 00:04 +0200, Pierre-Yves Chibon wrote:
>  On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > After the bugzilla upgrade this weekend a few things we run related to
>  > bugzilla are having issues: 
>  > 
>  > 1) on fas01, the script that sets fedorabugs and fedora_contrib on
>  > users in those fedora groups isn't working. 
>  > 
>  > 2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
>  > working. 
>  > 
>  > 3) pkgdb, which runs on app servers isnt' able to handle adding new
>  > packages. (we aren't sure what the error here is, but is likely bz
>  > related). 
>  
>  4) the review-stats script is having problems.
>  I spent some time on this and I have a patch that runs fine on my
>  machine against the official bugzilla and with the 'new' release of
>  python-bugzilla.
>  It is a little more heavy than the previous patches:
>  http://fpaste.org/dwwg/ 
And good news, it applies without problem to the script which is in
puppet (I worked on a clone of the fedora-infra's git).

Pierre

[Kevin Fenzi]

On Tue, 22 May 2012 00:04:33 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:

>  4) the review-stats script is having problems.
>  I spent some time on this and I have a patch that runs fine on my
>  machine against the official bugzilla and with the 'new' release of
>  python-bugzilla.
>  It is a little more heavy than the previous patches:
>  http://fpaste.org/dwwg/ 
Looks sane and if it works in your testing thats better than it's
current state. ;) 

+1 here. 

kevin

[Pierre-Yves Chibon]

On Tue, 2012-05-22 at 00:04 +0200, Pierre-Yves Chibon wrote:
>  On Mon, 2012-05-21 at 10:38 -0600, Kevin Fenzi wrote:
>  > After the bugzilla upgrade this weekend a few things we run related to
>  > bugzilla are having issues: 
>  > 
>  > 1) on fas01, the script that sets fedorabugs and fedora_contrib on
>  > users in those fedora groups isn't working. 
>  > 
>  > 2) on bapp01, the cron job that gathers easyfix bugzilla bugs is not
>  > working. 
>  > 
>  > 3) pkgdb, which runs on app servers isnt' able to handle adding new
>  > packages. (we aren't sure what the error here is, but is likely bz
>  > related). 
>  
>  4) the review-stats script is having problems.
>  I spent some time on this and I have a patch that runs fine on my
>  machine against the official bugzilla and with the 'new' release of
>  python-bugzilla.
>  It is a little more heavy than the previous patches:
>  http://fpaste.org/dwwg/ 
Actually, there was a typo left in the first version, so there is
(attached) a revised version.
The difference:
-    querydata['product'] = ['Fedora EPEL']
+    querydata['product'] = ['Fedora']

Fedora EPEL being used for testing as Fedora seems to always return a
ProxyError.

Pierre

[Kevin Fenzi]

On Tue, 22 May 2012 00:33:28 +0200
Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:

>  Actually, there was a typo left in the first version, so there is
>  (attached) a revised version.
>  The difference:
>  -    querydata['product'] = ['Fedora EPEL']
>  +    querydata['product'] = ['Fedora']
>  
>  Fedora EPEL being used for testing as Fedora seems to always return a
>  ProxyError.
>  
>  Pierre 
Good catch. +1 still. ;) 

kevin

[Seth Vidal]

On Mon, 21 May 2012 16:42:53 -0600
Kevin Fenzi <kevin(a)scrye.com&gt; wrote:

>  On Tue, 22 May 2012 00:33:28 +0200
>  Pierre-Yves Chibon <pingou(a)pingoured.fr&gt; wrote:
>  
>  > Actually, there was a typo left in the first version, so there is
>  > (attached) a revised version.
>  > The difference:
>  > -    querydata['product'] = ['Fedora EPEL']
>  > +    querydata['product'] = ['Fedora']
>  > 
>  > Fedora EPEL being used for testing as Fedora seems to always return
>  > a ProxyError.
>  > 
>  > Pierre
>  
>  Good catch. +1 still. ;) 
>   
+1 - been enjoying the cron email SO much.

:)
-sv