Ottomata has submitted this change and it was merged.
Change subject: Improve LVS check for Elasticsearch
......................................................................
Improve LVS check for Elasticsearch
Try to check the _nodes API to see if we can get a response for a
given node.
Bug: 57319
Change-Id: I315ec9b4d1bd19729f0bf45bcc7af20503068381
---
M modules/lvs/manifests/monitor.pp
M templates/icinga/checkcommands.cfg.erb
2 files changed, 6 insertions(+), 1 deletion(-)
Approvals:
Ottomata: Verified; Looks good to me, approved
diff --git a/modules/lvs/manifests/monitor.pp b/modules/lvs/manifests/monitor.pp
index 89a2b79..3050ab9 100644
--- a/modules/lvs/manifests/monitor.pp
+++ b/modules/lvs/manifests/monitor.pp
@@ -16,7 +16,7 @@
lvs::monitor_service_http { "ms-fe.pmtpa.wmnet": ip_address =>
"10.2.1.27", check_command =>
"check_http_lvs!ms-fe.pmtpa.wmnet!/monitoring/backend" }
lvs::monitor_service_http { "ms-fe.eqiad.wmnet": ip_address =>
"10.2.2.27", check_command =>
"check_http_lvs!ms-fe.eqiad.wmnet!/monitoring/backend" }
lvs::monitor_service_http { "parsoid.svc.eqiad.wmnet": ip_address =>
"10.2.2.28", check_command => "check_http_on_port!8000", contact_group =>
"admins,parsoid" }
- lvs::monitor_service_http { "search.svc.eqiad.wmnet": ip_address =>
"10.2.2.30", check_command => "check_http_on_port!9200", contact_group =>
"admins" }
+ lvs::monitor_service_http { "search.svc.eqiad.wmnet": ip_address =>
"10.2.2.30", check_command =>
"check_http_url_on_port_with_string!$host!9200!/_nodes/$host!$host",
contact_group => "admins" }
lvs::monitor_service_custom { "search-pool1.svc.eqiad.wmnet": ip_address
=> "10.2.2.11", port => 8123, description => "LVS Lucene", check_command =>
"check_lucene" }
lvs::monitor_service_custom { "search-pool2.svc.eqiad.wmnet": ip_address
=> "10.2.2.12", port => 8123, description => "LVS Lucene", check_command =>
"check_lucene" }
diff --git a/templates/icinga/checkcommands.cfg.erb
b/templates/icinga/checkcommands.cfg.erb
index ababda4..d2d1924 100644
--- a/templates/icinga/checkcommands.cfg.erb
+++ b/templates/icinga/checkcommands.cfg.erb
@@ -294,6 +294,11 @@
command_line $USER1$/check_http -H $ARG1$ -I $HOSTADDRESS$ -u $ARG2$
}
+define command{
+ command_name check_http_url_on_port_with_string
+ command_line $USER1$/check_http -H $ARG1$ -I $HOSTADDRESS$ -p $ARG2$
-u $ARG3$ -s $ARG4$
+ }
+
define command {
command_name check_https_url
command_line $USER1$/check_http -H $ARG1$ -S -I $HOSTADDRESS$ -u
$ARG2$
--
To view, visit https://gerrit.wikimedia.org/r/114399
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I315ec9b4d1bd19729f0bf45bcc7af20503068381
Gerrit-PatchSet: 8
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Chad <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: Chad <[email protected]>
Gerrit-Reviewer: Manybubbles <[email protected]>
Gerrit-Reviewer: Ori.livneh <[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
