BBlack has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/355219 )
Change subject: RPS cleanup 4/4: use new config file in puppet
......................................................................
RPS cleanup 4/4: use new config file in puppet
This allows future puppet usage of interface::rps to use
$rss_pattern again if warranted, via the config file, and also
sets up the class to automatically re-execute interface-rps on
relevant config and/or code changes.
Change-Id: Iad4a8522b74e05762164064c555200451fd5c2a8
---
M modules/interface/manifests/rps.pp
A modules/interface/templates/interface-rps-config.erb
2 files changed, 15 insertions(+), 7 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/19/355219/1
diff --git a/modules/interface/manifests/rps.pp
b/modules/interface/manifests/rps.pp
index a00060a..6430575 100644
--- a/modules/interface/manifests/rps.pp
+++ b/modules/interface/manifests/rps.pp
@@ -11,11 +11,14 @@
require interface::rpstools
require interface::rps::modparams
- if $rss_pattern != '' {
- $cmd = "/usr/local/sbin/interface-rps ${interface} ${rss_pattern}"
- }
- else {
- $cmd = "/usr/local/sbin/interface-rps ${interface}"
+ $cmd = "/usr/local/sbin/interface-rps ${interface}"
+ $cfg = "/etc/interface-rps.d/${interface}"
+
+ file { $cfg:
+ owner => 'root',
+ group => 'root',
+ mode => '0555',
+ content => template("${module_name}/interface-rps-config.erb"),
}
# Disable irqbalance
@@ -27,11 +30,14 @@
command => $cmd,
}
- # Exec immediately if newly-added
+ # Exec immediately on script or config change
exec { "rps-${interface}":
command => $cmd,
- subscribe => Augeas["${interface}_rps-${interface}"],
refreshonly => true,
+ subscribe => [
+ File['/usr/local/sbin/interface-rps'],
+ File[$cfg],
+ ],
}
}
diff --git a/modules/interface/templates/interface-rps-config.erb
b/modules/interface/templates/interface-rps-config.erb
new file mode 100644
index 0000000..8f66cd3
--- /dev/null
+++ b/modules/interface/templates/interface-rps-config.erb
@@ -0,0 +1,2 @@
+[Options]
+<% if @rss_pattern != '' %>rss_pattern = <%= @rss_pattern %><% end %>
--
To view, visit https://gerrit.wikimedia.org/r/355219
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iad4a8522b74e05762164064c555200451fd5c2a8
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
