BBlack has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/377443 )

Change subject: Add NUMA meminfo stats for cache+lvs nodes
......................................................................


Add NUMA meminfo stats for cache+lvs nodes

Bug: T175636
Change-Id: I045da8a3a102d48d58322e57c7267ffd959fd4c6
---
M hieradata/role/common/cache/canary.yaml
M hieradata/role/common/cache/misc.yaml
M hieradata/role/common/cache/text.yaml
M hieradata/role/common/cache/upload.yaml
M hieradata/role/common/lvs/balancer.yaml
5 files changed, 5 insertions(+), 0 deletions(-)

Approvals:
  BBlack: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/hieradata/role/common/cache/canary.yaml 
b/hieradata/role/common/cache/canary.yaml
index f2b29c0..174d27a 100644
--- a/hieradata/role/common/cache/canary.yaml
+++ b/hieradata/role/common/cache/canary.yaml
@@ -7,6 +7,7 @@
   - perf-roots
 prometheus::node_exporter::collectors_extra:
   - qdisc
+  - meminfo_numa
 # The contents of this hash control our DC->DC routing for varnish backend
 # daemons.  There should be a key for every cache datacenter.  The values must
 # be a core datacenter (eqiad or codfw), or at least must lead indirectly to
diff --git a/hieradata/role/common/cache/misc.yaml 
b/hieradata/role/common/cache/misc.yaml
index 834484e..1b2defb 100644
--- a/hieradata/role/common/cache/misc.yaml
+++ b/hieradata/role/common/cache/misc.yaml
@@ -3,6 +3,7 @@
 cache::lua_support: true
 prometheus::node_exporter::collectors_extra:
   - qdisc
+  - meminfo_numa
 # note this only affects tlsproxy now, should be moved to param there...
 cache::websocket_support: true
 # The contents of this hash control our DC->DC routing for varnish backend
diff --git a/hieradata/role/common/cache/text.yaml 
b/hieradata/role/common/cache/text.yaml
index 2741c01..9e6417c 100644
--- a/hieradata/role/common/cache/text.yaml
+++ b/hieradata/role/common/cache/text.yaml
@@ -4,6 +4,7 @@
   - perf-roots
 prometheus::node_exporter::collectors_extra:
   - qdisc
+  - meminfo_numa
 # The contents of this hash control our DC->DC routing for varnish backend
 # daemons.  There should be a key for every cache datacenter.  The values must
 # be a core datacenter (eqiad or codfw), or at least must lead indirectly to
diff --git a/hieradata/role/common/cache/upload.yaml 
b/hieradata/role/common/cache/upload.yaml
index 3b0a21a..13d2b2e 100644
--- a/hieradata/role/common/cache/upload.yaml
+++ b/hieradata/role/common/cache/upload.yaml
@@ -4,6 +4,7 @@
   - perf-roots
 prometheus::node_exporter::collectors_extra:
   - qdisc
+  - meminfo_numa
 cache::tune_for_media: true
 # The contents of this hash control our DC->DC routing for varnish backend
 # daemons.  There should be a key for every cache datacenter.  The values must
diff --git a/hieradata/role/common/lvs/balancer.yaml 
b/hieradata/role/common/lvs/balancer.yaml
index cac5133..00628e8 100644
--- a/hieradata/role/common/lvs/balancer.yaml
+++ b/hieradata/role/common/lvs/balancer.yaml
@@ -4,5 +4,6 @@
   listen_address: "%{::ipaddress}"
 prometheus::node_exporter::collectors_extra:
   - ipvs
+  - meminfo_numa
 profile::pybal::config_source: etcd
 profile::pybal::bgp: "yes"

-- 
To view, visit https://gerrit.wikimedia.org/r/377443
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I045da8a3a102d48d58322e57c7267ffd959fd4c6
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <bbl...@wikimedia.org>
Gerrit-Reviewer: BBlack <bbl...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to