From 964dbc45beca9813f6f8712775e9fb598091c744 Mon Sep 17 00:00:00 2001 From: Jeremy Stanley Date: Mon, 6 May 2013 17:52:21 +0000 Subject: [PATCH] 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 Reviewed-by: Clark Boylan Approved: Jeremy Stanley Reviewed-by: Jeremy Stanley Tested-by: Jenkins --- modules/reviewday/manifests/site.pp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/reviewday/manifests/site.pp b/modules/reviewday/manifests/site.pp index 7edf50f659..83814c2222 100644 --- a/modules/reviewday/manifests/site.pp +++ b/modules/reviewday/manifests/site.pp @@ -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', }