opendev
/
puppet-reviewday
Code Issues Proposed changes

Decrease reviewday update frequency. 

* modules/reviewday/manifests/site.pp: The frequency of the update
reviewday cron job is now every half hour instead of every quarter
hour. Additionally, a lock is added to prevent overlapping runs in
case the script begins to take longer. This is needed since recent
Launchpad API changes have caused it to perform considerably slower
than before.

Change-Id: I9fcdbb350a61ab29200068406ccb8b8eaa5e4244
Reviewed-on: https://review.openstack.org/28331
Reviewed-by: Dan Prince <dprince@redhat.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
This commit is contained in:
Jeremy Stanley 2013-05-06 17:52:21 +00:00 committed by Jenkins
parent cafa6ead92
commit 54e3aaf95f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/site.pp
View File

@ -98,8 +98,8 @@ define reviewday::site(
}
cron { 'update reviewday':
command => "cd /var/lib/reviewday/reviewday && PYTHONPATH=\$PWD python bin/reviewday -o ${httproot}",
minute => '*/15',
command => "cd /var/lib/reviewday/reviewday && PYTHONPATH=\$PWD flock -n /var/lib/reviewday/update.lock python bin/reviewday -o ${httproot}",
minute => '*/30',
user => 'reviewday',
}