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

Delta Between Two Patch Sets: modules/web/manifests/server.pp

Issue 29396643: NoIssue - Reconfigure Mercurial pull cron-jobs (Closed)
Left Patch Set: Add a missing cronjob Created March 29, 2017, 12:06 a.m.
Right Patch Set: For comment #4 Created March 29, 2017, 12:13 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « modules/sitescripts/manifests/init.pp ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 class web::server( 1 class web::server(
2 $vhost, 2 $vhost,
3 $repository, 3 $repository,
4 $certificate = hiera('web::server::certificate', 'undef'), 4 $certificate = hiera('web::server::certificate', 'undef'),
5 $private_key = hiera('web::server::private_key', 'undef'), 5 $private_key = hiera('web::server::private_key', 'undef'),
6 $is_default = false, 6 $is_default = false,
7 $aliases = undef, 7 $aliases = undef,
8 $custom_config = undef, 8 $custom_config = undef,
9 $multiplexer_locations = undef, 9 $multiplexer_locations = undef,
10 $geoip = false, 10 $geoip = false,
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 "/var/www/docs", 125 "/var/www/docs",
126 ]: 126 ]:
127 ensure => directory, 127 ensure => directory,
128 owner => www, 128 owner => www,
129 mode => 755, 129 mode => 755,
130 } 130 }
131 131
132 cron {'update_cms': 132 cron {'update_cms':
133 ensure => present, 133 ensure => present,
134 command => "hg pull -q -u -R /opt/cms", 134 command => "hg pull -q -u -R /opt/cms",
135 minute => '5-55/20', 135 minute => '4-59/20',
mathias 2017/03/29 00:10:55 Why -55 when all similar ones use -59?
136 } 136 }
137 137
138 cron {'update_repo': 138 cron {'update_repo':
139 ensure => present, 139 ensure => present,
140 command => "hg pull -q -R /home/www/${repository} && python -m cms.bin.gener ate_static_pages /home/www/${repository} /var/www/${vhost}", 140 command => "hg pull -q -R /home/www/${repository} && python -m cms.bin.gener ate_static_pages /home/www/${repository} /var/www/${vhost}",
141 user => www, 141 user => www,
142 minute => '4-59/20', 142 minute => '5-59/20',
mathias 2017/03/29 00:10:55 This should probably happen in the minute after up
143 } 143 }
144 144
145 } 145 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld