Dzahn has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/380901 )
Change subject: icinga/base: re-enable screen/tmux monitoring
..
icinga/base: re-enable screen/tmux monitoring
Re-enabling screen monitoring again after it was initially
reverted in I959702d4c12b. But now 2 precautions have been taken:
a) several hosts have been whitelisted in I8fc231cbcf2f60717046
b) CRIT threshold initially raised to a full year, so only WARNS
unless a screen actually runs over a year. Ied897edabbe1237ad
We can then decide if we want to whitelist more hosts and how much
to lower the threshold based on the warning list.
Since they will be just warnings they will not spam IRC and should
just show up in Icinga web UI.
Bug: T165348
Change-Id: I0569c449294603c81ff16fa34ab95a2c7fc5906b
---
M modules/base/manifests/monitoring/host.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/base/manifests/monitoring/host.pp
b/modules/base/manifests/monitoring/host.pp
index 91733b3..966307b 100644
--- a/modules/base/manifests/monitoring/host.pp
+++ b/modules/base/manifests/monitoring/host.pp
@@ -147,7 +147,7 @@
}
}
-if hiera('monitor_screens', false) {
+if hiera('monitor_screens', true) {
file { '/usr/local/lib/nagios/plugins/check_long_procs':
ensure => present,
--
To view, visit https://gerrit.wikimedia.org/r/380901
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0569c449294603c81ff16fa34ab95a2c7fc5906b
Gerrit-PatchSet: 4
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn
Gerrit-Reviewer: Alexandros Kosiaris
Gerrit-Reviewer: Dzahn
Gerrit-Reviewer: Gehel
Gerrit-Reviewer: Jcrespo
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits