Linux Archive

Linux Archive (http://www.linux-archive.org/)
-   Fedora Infrastructure (http://www.linux-archive.org/fedora-infrastructure/)
-   -   Yet another freeze break: reviewstatus (http://www.linux-archive.org/fedora-infrastructure/670737-yet-another-freeze-break-reviewstatus.html)

Kevin Fenzi 05-24-2012 09:45 PM

Yet another freeze break: reviewstatus
 
I'd like to revert puppet commits:

55103ec6ce3a80234391229f6b10e220fa13e72d
5b131d0e489f5349d5cef6a8e0ffdc2febda0441

This reverts review-stats.py to the version that we had before monday,
which was working with the old python-bugzilla (which we have already
reverted to).

I've tested it locally and it does only make 6 bugzilla calls when used
with the python-bugzilla version we have reverted to, so we shouldn't
be causing undue load on bugzilla.

I'd also like to:

diff --git a/modules/review-stats/manifests/init.pp b/modules/review-stats/manifests/init.pp
index 9a016a5..0376a77 100644
--- a/modules/review-stats/manifests/init.pp
+++ b/modules/review-stats/manifests/init.pp
@@ -57,7 +57,7 @@ class review-stats::build inherits review-stats {
command => "/usr/bin/python /usr/local/bin/pyGetReviewByFlags.py -t /usr/local/share/review-stats -d
user => "apache",
minute => [ 0, 30 ],
- ensure => "absent",
+ ensure => "present",
}

}

to re-enable it.

+1s?

kevin
_______________________________________________
infrastructure mailing list
infrastructure@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

Toshio Kuratomi 05-24-2012 10:02 PM

Yet another freeze break: reviewstatus
 
On Thu, May 24, 2012 at 03:45:02PM -0600, Kevin Fenzi wrote:
> I'd like to revert puppet commits:
>
> 55103ec6ce3a80234391229f6b10e220fa13e72d
> 5b131d0e489f5349d5cef6a8e0ffdc2febda0441
>
> This reverts review-stats.py to the version that we had before monday,
> which was working with the old python-bugzilla (which we have already
> reverted to).
>
> I've tested it locally and it does only make 6 bugzilla calls when used
> with the python-bugzilla version we have reverted to, so we shouldn't
> be causing undue load on bugzilla.
>
> I'd also like to:
>
> diff --git a/modules/review-stats/manifests/init.pp b/modules/review-stats/manifests/init.pp
> index 9a016a5..0376a77 100644
> --- a/modules/review-stats/manifests/init.pp
> +++ b/modules/review-stats/manifests/init.pp
> @@ -57,7 +57,7 @@ class review-stats::build inherits review-stats {
> command => "/usr/bin/python /usr/local/bin/pyGetReviewByFlags.py -t /usr/local/share/review-stats -d
> user => "apache",
> minute => [ 0, 30 ],
> - ensure => "absent",
> + ensure => "present",
> }
>
> }
>
> to re-enable it.
>
> +1s?
>
+1

-Toshio
_______________________________________________
infrastructure mailing list
infrastructure@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/infrastructure


All times are GMT. The time now is 12:15 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.