Ottomata has submitted this change and it was merged.

Change subject: Fix include and thresholds for icinga alerts for confluent 
kafka brokers
......................................................................


Fix include and thresholds for icinga alerts for confluent kafka brokers

Bug: T121562
Change-Id: I7062da55b377c2de9f3fe7d8976b7a9f5d38134a
---
M modules/role/manifests/kafka/analytics/broker.pp
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  Ottomata: Verified; Looks good to me, approved



diff --git a/modules/role/manifests/kafka/analytics/broker.pp 
b/modules/role/manifests/kafka/analytics/broker.pp
index 9d64b7f..e8bd180 100644
--- a/modules/role/manifests/kafka/analytics/broker.pp
+++ b/modules/role/manifests/kafka/analytics/broker.pp
@@ -109,8 +109,11 @@
 
         # Monitor kafka in production
         if $::realm == 'production' {
-            class { '::confluent::kafka::server::alerts':
-                nagios_servicegroup => "analytics_${::site}",
+            class { '::confluent::kafka::broker::alerts':
+                nagios_servicegroup     => "analytics_${::site}",
+                # TODO: tweak these
+                replica_maxlag_warning  => '1000000',
+                replica_maxlag_critical => '5000000',
             }
         }
     }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7062da55b377c2de9f3fe7d8976b7a9f5d38134a
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to