Asher has submitted this change and it was merged.
Change subject: cpufrequtils shouldn't be treated as a service, except to start
at boot
......................................................................
cpufrequtils shouldn't be treated as a service, except to start at boot
Change-Id: Iad5c63ed5ad6cde5f3299a7768d0c31df64abd99
---
M modules/cpufrequtils/manifests/init.pp
1 file changed, 10 insertions(+), 5 deletions(-)
Approvals:
Asher: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/cpufrequtils/manifests/init.pp
b/modules/cpufrequtils/manifests/init.pp
index 8bfcc51..03988b0 100644
--- a/modules/cpufrequtils/manifests/init.pp
+++ b/modules/cpufrequtils/manifests/init.pp
@@ -13,15 +13,20 @@
ensure => present;
}
+ # start at boot
+ service { 'cpufrequtils':
+ enable => true,
+ require => Package['cpufrequtils'],
+ }
+
file { '/etc/default/cpufrequtils':
content => "GOVERNOR=${governor}\n",
+ notify => Exec['apply cpufrequtils'],
require => Package['cpufrequtils'];
}
- service { 'cpufrequtils':
- enable => true,
- hasstatus => false,
- require => Package['cpufrequtils'],
- subscribe => File['/etc/default/cpufrequtils'];
+ exec { 'apply cpufrequtils':
+ command => '/etc/init.d/cpufrequtils start',
+ refreshonly => true
}
}
--
To view, visit https://gerrit.wikimedia.org/r/57667
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iad5c63ed5ad6cde5f3299a7768d0c31df64abd99
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Asher <afeld...@wikimedia.org>
Gerrit-Reviewer: Asher <afeld...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
