Dzahn has submitted this change and it was merged. Change subject: icinga-wm/ircecho: do not set variables in node ......................................................................
icinga-wm/ircecho: do not set variables in node do not set those variables directly in site.pp on the 'neon' node. move them to the role class. add a comment what this is actually for Change-Id: I6e5d832c925178a7baae1cdc436efe25dd2c9d13 --- M manifests/role/echoirc.pp M manifests/site.pp 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: jenkins-bot: Verified Dzahn: Looks good to me, approved diff --git a/manifests/role/echoirc.pp b/manifests/role/echoirc.pp index 535eb93..801e453 100644 --- a/manifests/role/echoirc.pp +++ b/manifests/role/echoirc.pp @@ -1,4 +1,10 @@ +# role class for Icinga IRC bot class role::echoirc { + + $ircecho_logs = { '/var/log/icinga/irc.log' => '#wikimedia-operations' } + $ircecho_nick = 'icinga-wm' + $ircecho_server = 'chat.freenode.net' + include ircecho system::role { 'ircecho': description => 'ircecho server' } @@ -8,5 +14,6 @@ description => 'ircecho_service_running', nrpe_command => '/usr/lib/nagios/plugins/check_procs -w 1:4 -c 1:20 -a ircecho', } + } diff --git a/manifests/site.pp b/manifests/site.pp index 6366613..0ae3e52 100644 --- a/manifests/site.pp +++ b/manifests/site.pp @@ -1840,9 +1840,6 @@ class { 'base::firewall': } $domain_search = 'wikimedia.org pmtpa.wmnet eqiad.wmnet esams.wikimedia.org' - $ircecho_logs = { '/var/log/icinga/irc.log' => '#wikimedia-operations' } - $ircecho_nick = 'icinga-wm' - $ircecho_server = 'chat.freenode.net' include standard include icinga::monitor include role::ishmael -- To view, visit https://gerrit.wikimedia.org/r/129662 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e5d832c925178a7baae1cdc436efe25dd2c9d13 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits