Yet another freeze break: reviewstatus

[Stephen John Smoogen]

On 24 May 2012 16:02, Toshio Kuratomi <a.badger at gmail.com> wrote:
> 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

+1

-- 
Stephen J Smoogen.
"The core skill of innovators is error recovery, not failure avoidance."
Randy Nelson, President of Pixar University.
"Years ago my mother used to say to me,... Elwood, you must be oh
so smart or oh so pleasant. Well, for years I was smart. I
recommend pleasant. You may quote me."  —James Stewart as Elwood P. Dowd