search

braintree / litmus_paper

Backend health tester for HA Services
MIT License
32 stars 15 forks source link

Haproxy will report a backend's status as 'UP' when there are no #24

Closed bjhaid closed 7 years ago

bjhaid commented 7 years ago

backend configured, this PR uses backend 'weight' instead of backend 'status' from the haproxy stats socket to represent a backend's state as this represents the total weights of the backends and will be 0 when there's no backend configured.

lollipopman commented 7 years ago

looks good, can you add note in the commit message that 'status' will report UP when there are no backends

lollipopman commented 7 years ago

missed the header, this looks good :+1:

bjhaid commented 7 years ago

The error in 1.9.3 travis is a bug with the version of bundler used:

https://github.com/bundler/bundler/issues/3558

We can include installing a newer version of bundler in before_install but I'll merge this PR regardless tomorrow