Page MenuHomePhabricator

CirrusSearch: Reinvestigate LVS pool check
Closed, DeclinedPublic

Description

Reinvestigate LVS pool check command. Right now we're pretty much just making sure the thing responds to http. Maybe we can do better?


Version: unspecified
Severity: normal

Details

Reference
bz57319

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:28 AM
bzimport added a project: CirrusSearch.
bzimport set Reference to bz57319.

Change 114399 had a related patch set uploaded by Chad:
WIP: Improve LVS check for Elasticsearch

https://gerrit.wikimedia.org/r/114399

Change 114399 merged by Ottomata:
Improve LVS check for Elasticsearch

https://gerrit.wikimedia.org/r/114399

I've investigated this today... We have monitoring of the Elasticsearch processes via Icinga anyway, and I think the added complication to LVS wouldn't be worth it. If a node was failing I assume it wouldn't even be responding on 9200, so it's probably enough as it is.

Marking WONTFIX.