Paladox has uploaded a new change for review. https://gerrit.wikimedia.org/r/324741
Change subject: udp2log: Replace undefined variable with $ensure_monitor_processes ...................................................................... udp2log: Replace undefined variable with $ensure_monitor_processes Replaces $ensure_monitor_processes with $monitor_processes Fix is suggested by scfc in T152104 Follows-Up: 90fc2e11b3618545523d022d54df022d50bd3858 Bug: T152104 Change-Id: I59c591767361d048749ba424ef0e0abd1db4f9a5 --- M modules/udp2log/manifests/instance/monitoring.pp 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/41/324741/1 diff --git a/modules/udp2log/manifests/instance/monitoring.pp b/modules/udp2log/manifests/instance/monitoring.pp index 4839300..4c41a47 100644 --- a/modules/udp2log/manifests/instance/monitoring.pp +++ b/modules/udp2log/manifests/instance/monitoring.pp @@ -39,7 +39,7 @@ # /etc/udp2log/$name is running # FIXME - top-scope var without namespace, will break in puppet 2.8 # lint:ignore:variable_scope - if ($ensure_monitor_processes == true and $ensure == 'running') { + if ($monitor_processes == true and $ensure == 'running') { # lint:endignore nrpe::monitor_service { "udp2log_procs-${name}": ensure => 'present', -- To view, visit https://gerrit.wikimedia.org/r/324741 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I59c591767361d048749ba424ef0e0abd1db4f9a5 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits