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

Unified Diff: modules/statsmaster/manifests/init.pp

Issue 29326111: Issue 3010 - Introduce class geoip (Closed)
Patch Set: Created Sept. 8, 2015, 12:11 p.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/statsmaster/files/cron_geoipdb_update.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/statsmaster/manifests/init.pp
diff --git a/modules/statsmaster/manifests/init.pp b/modules/statsmaster/manifests/init.pp
index c8553413312f5d81fc3190e525c57925ae9371a2..030dc8137f53bec9e602fa45c8b7f4962b8be22d 100644
--- a/modules/statsmaster/manifests/init.pp
+++ b/modules/statsmaster/manifests/init.pp
@@ -64,21 +64,17 @@ class statsmaster(
ensure => absent,
}
- file {'/opt/cron_geoipdb_update.py':
- ensure => file,
- owner => root,
- mode => 0750,
- source => 'puppet:///modules/statsmaster/cron_geoipdb_update.py',
+ cron {'geoipdb_update':
+ ensure => 'absent',
}
- cron {'geoipdb_update':
- ensure => present,
- require => File['/opt/cron_geoipdb_update.py'],
- command => '/opt/cron_geoipdb_update.py',
- environment => ['MAILTO=admins@adblockplus.org,root'],
- user => root,
- hour => 3,
- minute => 15,
- monthday => 3,
+ class {'geoip':
+ cron => {
+ 'environment' => ['MAILTO=admins@adblockplus.org,root'],
+ 'hour' => 3,
+ 'minute' => 15,
+ 'monthday' => 3,
+ },
+ script => '/opt/cron_geoipdb_update.py',
}
}
« no previous file with comments | « modules/statsmaster/files/cron_geoipdb_update.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld