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

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

Issue 29338473: Issue 3574 - Move cron setup from class base to module adblockplus (Closed)
Patch Set: Created March 16, 2016, 9:49 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 | « no previous file | modules/base/manifests/init.pp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/manifests/init.pp
diff --git a/modules/adblockplus/manifests/init.pp b/modules/adblockplus/manifests/init.pp
index f163f88446e13a60cfac472a83f906abe96884a0..2f1dfdc8fb0f8e9ece0b5cb3c7f823c262bd25d6 100644
--- a/modules/adblockplus/manifests/init.pp
+++ b/modules/adblockplus/manifests/init.pp
@@ -76,6 +76,13 @@ class adblockplus (
notify => Service['cron'];
}
+ # Explicit resource required only to ensure cron(8) is running;
+ # there is no real requirement for a rationship with another resource
+ service {'cron':
+ ensure => 'running',
+ enable => true,
+ }
+
# Work around https://issues.adblockplus.org/ticket/3479
if $::environment == 'development' {
« no previous file with comments | « no previous file | modules/base/manifests/init.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld