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

Side by Side Diff: modules/statsmaster/manifests/awstats.pp

Issue 29366977: Issue 1503 - Remove hard-coded $MAILTO variable from stats cron environment (Closed)
Patch Set: Created Dec. 7, 2016, 7:58 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | modules/statsmaster/manifests/downloads.pp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 class statsmaster::awstats { 1 class statsmaster::awstats {
2 package {['awstats', 'libgeo-ip-perl']:} 2 package {['awstats', 'libgeo-ip-perl']:}
3 3
4 file {['/var/www/awstatsdata', '/var/www/awstatsdatadaily', 4 file {['/var/www/awstatsdata', '/var/www/awstatsdatadaily',
5 '/var/www/awstatsconf', '/var/www/awstats', 5 '/var/www/awstatsconf', '/var/www/awstats',
6 '/var/www/awstats/archive', '/var/www/awstats/daily']: 6 '/var/www/awstats/archive', '/var/www/awstats/daily']:
7 ensure => directory, 7 ensure => directory,
8 owner => root, 8 owner => root,
9 mode => 0755, 9 mode => 0755,
10 } 10 }
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 140
141 cron {'awstats_update': 141 cron {'awstats_update':
142 ensure => present, 142 ensure => present,
143 require => [ 143 require => [
144 Package['awstats', 'libgeo-ip-perl'], 144 Package['awstats', 'libgeo-ip-perl'],
145 File['/home/stats/process_logs', '/home/stats/build_static', 145 File['/home/stats/process_logs', '/home/stats/build_static',
146 '/var/www/awstatsdata', '/var/www/awstatsdatadaily', 146 '/var/www/awstatsdata', '/var/www/awstatsdatadaily',
147 '/var/www/awstats', '/var/www/awstatsconf'], 147 '/var/www/awstats', '/var/www/awstatsconf'],
148 ], 148 ],
149 command => '/home/stats/process_logs && /home/stats/build_static', 149 command => '/home/stats/process_logs && /home/stats/build_static',
150 environment => ['MAILTO=admins@adblockplus.org,root'],
151 user => stats, 150 user => stats,
152 hour => 4, 151 hour => 4,
153 minute => 0, 152 minute => 0,
154 } 153 }
155 154
156 cron {'awstats_prevmonth': 155 cron {'awstats_prevmonth':
157 ensure => present, 156 ensure => present,
158 require => [ 157 require => [
159 Package['awstats'], 158 Package['awstats'],
160 File['/home/stats/build_static', '/home/stats/anonymize_ips', 159 File['/home/stats/build_static', '/home/stats/anonymize_ips',
161 '/var/www/awstatsdata', '/var/www/awstatsdatadaily', 160 '/var/www/awstatsdata', '/var/www/awstatsdatadaily',
162 '/var/www/awstatsconf', '/var/www/awstats'], 161 '/var/www/awstatsconf', '/var/www/awstats'],
163 ], 162 ],
164 command => '/home/stats/anonymize_ips prevmonth && /home/stats/build_static prevmonth', 163 command => '/home/stats/anonymize_ips prevmonth && /home/stats/build_static prevmonth',
165 environment => ['MAILTO=admins@adblockplus.org,root'],
166 user => stats, 164 user => stats,
167 monthday => 1, 165 monthday => 1,
168 hour => 6, 166 hour => 6,
169 minute => 0, 167 minute => 0,
170 } 168 }
171 } 169 }
OLDNEW
« no previous file with comments | « no previous file | modules/statsmaster/manifests/downloads.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld