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

Delta Between Two Patch Sets: modules/statsclient/manifests/init.pp

Issue 11468051: Update stats processing (Closed)
Left Patch Set: Using PyPy for log processing Created Aug. 29, 2013, 8:07 p.m.
Right Patch Set: Dropped Jinja2 requirement Created Aug. 29, 2013, 9:06 p.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/statsclient/files/sitescripts.ini ('k') | modules/statsmaster/files/stats.adblockplus.org » ('j') | 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 statsclient ( 1 class statsclient (
2 $log_path, 2 $log_path,
3 $custom_sitescriptsini_source = [], 3 $custom_sitescriptsini_source = [],
4 ) { 4 ) {
5 5
6 $sitescriptsini_source = flatten(['puppet:///modules/statsclient/sitescripts.i ni', $custom_sitescriptsini_source]) 6 $sitescriptsini_source = flatten(['puppet:///modules/statsclient/sitescripts.i ni', $custom_sitescriptsini_source])
7 7
8 user {'stats': 8 user {'stats':
9 ensure => present, 9 ensure => present,
10 home => '/home/stats', 10 home => '/home/stats',
(...skipping 20 matching lines...) Expand all
31 X11Forwarding no 31 X11Forwarding no
32 AllowAgentForwarding no 32 AllowAgentForwarding no
33 GatewayPorts no 33 GatewayPorts no
34 ForceCommand cat /var/www/stats.json', 34 ForceCommand cat /var/www/stats.json',
35 } 35 }
36 36
37 class {'sitescripts': 37 class {'sitescripts':
38 sitescriptsini_source => $sitescriptsini_source, 38 sitescriptsini_source => $sitescriptsini_source,
39 } 39 }
40 40
41 package {['pypy', 'python-jinja2']:} 41 package {'pypy':}
Felix Dahlke 2013/08/29 20:23:25 Since we're using PyPy now, don't we need Jinja2 f
Wladimir Palant 2013/08/29 21:02:31 Yes :-( I realized that this only worked because I
42 42
43 file {'/var/www/stats.json': 43 file {'/var/www/stats.json':
44 ensure => present, 44 ensure => present,
45 owner => stats, 45 owner => stats,
46 mode => 644, 46 mode => 644,
47 } 47 }
48 48
49 file {'/opt/cron_geoipdb_update.sh': 49 file {'/opt/cron_geoipdb_update.sh':
50 ensure => file, 50 ensure => file,
51 owner => root, 51 owner => root,
(...skipping 18 matching lines...) Expand all
70 cron {'geoipdb_update': 70 cron {'geoipdb_update':
71 ensure => present, 71 ensure => present,
72 require => File['/opt/cron_geoipdb_update.sh'], 72 require => File['/opt/cron_geoipdb_update.sh'],
73 command => '/opt/cron_geoipdb_update.sh', 73 command => '/opt/cron_geoipdb_update.sh',
74 user => root, 74 user => root,
75 hour => 3, 75 hour => 3,
76 minute => 15, 76 minute => 15,
77 monthday => 3, 77 monthday => 3,
78 } 78 }
79 } 79 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld