Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/web/manifests/server.pp

Issue 29396643: NoIssue - Reconfigure Mercurial pull cron-jobs (Closed)
Patch Set: Add a missing cronjob Created March 29, 2017, 12:06 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « modules/sitescripts/manifests/init.pp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/web/manifests/server.pp
===================================================================
--- a/modules/web/manifests/server.pp
+++ b/modules/web/manifests/server.pp
@@ -132,14 +132,14 @@
cron {'update_cms':
ensure => present,
command => "hg pull -q -u -R /opt/cms",
- minute => '5-55/10',
+ minute => '5-55/20',
mathias 2017/03/29 00:10:55 Why -55 when all similar ones use -59?
}
cron {'update_repo':
ensure => present,
command => "hg pull -q -R /home/www/${repository} && python -m cms.bin.generate_static_pages /home/www/${repository} /var/www/${vhost}",
user => www,
- minute => '*/10',
+ minute => '4-59/20',
mathias 2017/03/29 00:10:55 This should probably happen in the minute after up
}
}
« no previous file with comments | « modules/sitescripts/manifests/init.pp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld