Lcarr has submitted this change and it was merged.
Change subject: Revert "Adding icinga check to make sure kafka brokers aren't
getting too many produce requests"
......................................................................
Revert "Adding icinga check to make sure kafka brokers aren't getting too many
produce requests"
This reverts commit 6e626a9dc32d7d99d2070844315a0b04e03865da.
Change-Id: I87c3ad44806db2d549940e8702e791d8692d2893
---
M manifests/misc/analytics.pp
M templates/icinga/checkcommands.cfg.erb
2 files changed, 6 insertions(+), 18 deletions(-)
Approvals:
Lcarr: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/misc/analytics.pp b/manifests/misc/analytics.pp
index c190eea..cf598fa 100644
--- a/manifests/misc/analytics.pp
+++ b/manifests/misc/analytics.pp
@@ -29,16 +29,10 @@
class misc::analytics::monitoring::kafka::server {
# Set up icinga monitoring of Kafka broker server produce requests per
second.
# If this drops too low, trigger an alert
- monitor_service { 'kafka_broker-ProduceRequestsPerSecond':
- description =>
'kafka_network_SocketServerStats.ProduceRequestsPerSecond',
- check_command => 'check_kafka_broker_produce_requests_min!2!1',
- contact_group => 'analytics',
- }
- # Set up icinga monitoring of Kafka broker server produce requests per
second.
- # If this drops too high, trigger an alert
- monitor_service { 'kafka_broker-ProduceRequestsPerSecond':
- description =>
'kafka_network_SocketServerStats.ProduceRequestsPerSecond',
- check_command =>
'check_kafka_broker_produce_requests_max!15!20',
- contact_group => 'analytics',
+ # for this udp2log instance.
+ monitor_service { "kakfa-broker-ProduceRequestsPerSecond":
+ description =>
"kafka_network_SocketServerStats.ProduceRequestsPerSecond",
+ check_command =>
"check_kafka_broker_produce_requests!2!1",
+ contact_group => "analytics",
}
}
diff --git a/templates/icinga/checkcommands.cfg.erb
b/templates/icinga/checkcommands.cfg.erb
index b40ea35..b156f7e 100644
--- a/templates/icinga/checkcommands.cfg.erb
+++ b/templates/icinga/checkcommands.cfg.erb
@@ -617,14 +617,8 @@
# Analytics Kraken checks
# Check that Kafka Brokers are getting ProduceRequests.
define command{
- command_name check_kafka_broker_produce_requests_min
+ command_name check_kafka_broker_produce_requests
command_line $USER1$/check_ganglios_generic_value -H $HOSTADDRESS$
-m kafka_network_SocketServerStats.ProduceRequestsPerSecond -w $ARG1$ -c $ARG2$
-o lt
-}
-
-# Check that Kafka Brokers aren't getting TOO many ProduceRequests
-define command{
- command_name check_kafka_producer_produce_requests_max
- command_line $USER1$/check_ganglios_generic_value -H $HOSTADDRESS$
-m kafka_network_SocketServerStats.ProduceRequestsPerSecond -w $ARG1$ -c $ARG2$
-o gt
}
# Alerts for data loss in Kraken HDFS.
--
To view, visit https://gerrit.wikimedia.org/r/72052
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I87c3ad44806db2d549940e8702e791d8692d2893
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Lcarr <[email protected]>
Gerrit-Reviewer: Lcarr <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
