Krinkle has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/369835 )
Change subject: webperf: Move statsv monitoring from 'webperf' to
'webperf::statsv'
......................................................................
webperf: Move statsv monitoring from 'webperf' to 'webperf::statsv'
The base class is just for the user/group/directory, it doesn't
make sense for a specific service's monitoring to be ensured for
any class (e.g. webperf::navtiming) that wants to run a webperf
process.
It all works out now because the three webperf services run on
the same node, but this may be changing.
Bug: T158837
Change-Id: Icf006f551ac6ae398fb543dd9f50e25d495dec45
---
M modules/webperf/manifests/init.pp
M modules/webperf/manifests/statsv.pp
2 files changed, 7 insertions(+), 7 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/35/369835/1
diff --git a/modules/webperf/manifests/init.pp
b/modules/webperf/manifests/init.pp
index dfc5262..a6deb91 100644
--- a/modules/webperf/manifests/init.pp
+++ b/modules/webperf/manifests/init.pp
@@ -1,7 +1,7 @@
# == Class: webperf
#
-# This Puppet module provisions a set of client-side performance
-# monitoring scripts for Wikimedia sites.
+# This base class provides a user, group, and working directory for
+# webperf processes.
#
class webperf {
group { 'webperf':
@@ -19,10 +19,5 @@
file { '/srv/webperf':
ensure => directory,
- }
-
- nrpe::monitor_service { 'statsv':
- description => 'statsv process',
- nrpe_command => '/usr/lib/nagios/plugins/check_procs -c 1: -C python
-a statsv',
}
}
diff --git a/modules/webperf/manifests/statsv.pp
b/modules/webperf/manifests/statsv.pp
index 2ff2763..320c045 100644
--- a/modules/webperf/manifests/statsv.pp
+++ b/modules/webperf/manifests/statsv.pp
@@ -28,4 +28,9 @@
provider => 'systemd',
subscribe => File['/lib/systemd/system/statsv.service'],
}
+
+ nrpe::monitor_service { 'statsv':
+ description => 'statsv process',
+ nrpe_command => '/usr/lib/nagios/plugins/check_procs -c 1: -C python
-a statsv',
+ }
}
--
To view, visit https://gerrit.wikimedia.org/r/369835
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf006f551ac6ae398fb543dd9f50e25d495dec45
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Krinkle <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
