openresty / lua-resty-upstream-healthcheck

Health Checker for Nginx Upstream Servers in Pure Lua
521 stars 134 forks source link

Added all_down flag #52

Open eglinux opened 6 years ago

eglinux commented 6 years ago

Added new all_down flag (upstream:all_down) to the global dict structure. Needed to detect situations when all peers are down for the upstream. If such has happened - we switch all peers to up state instead. Proper error message will be appearing in the error log.

Related to the Issue #51

eglinux commented 6 years ago

Looks like TEST 11 is failing because there is only one peer in upstream foo.com and it is actually down.

So, if this request has to be accepted I think test can be modified to expect status 'up' instead of 'DOWN'. Since it is the point to achieve this.

eglinux commented 6 years ago

Any chance that you would evaluate the changes ? )