Hi,

It is a bug.

the ngx_upstream_get_peer only check the index greater than i; forgot to
check i itself.

I used my nginx patch for healthcheck, I have used it in production more
than half a year. I will upload it to my github in some hours.

liseen

On Fri, Sep 23, 2011 at 4:34 AM, Sjaak Pieterse <sjaak23@gmail.com> wrote:

> Hi there,
>
> i'm trying to use the HttpHealthcheckModule for nginx, but i have some
> troubles with it.
>
> i have two servers in my upstream, when sabotaging the health for one
> server i see in the status view of healthcheck that the server is
> down(1), but if i go to the website i'm checking i still come out on
> it and see a broken page.
>
> how can i arrange that the server automatically is marked as down when
> the check fails?
>
> sorry for my bad english and maybe noob questions.
>
> config:
> upstream www-health{
> server x.x.x.1 ;
> server x.x.x.2 ;
> healthcheck_enabled;
> healthcheck_delay 10000 ;
> healthcheck_timeout 1000;
> healthcheck_failcount 2;
> #healthcheck_expected 'I_AM_ALIVE';
> #Important: HTTP/1.0
> healthcheck_send "GET / HTTP/1.0" 'Host: health.test.x.com'
> 'Conection: close' ;
> }
>
> nginx: nginx version: nginx/1.0.6
> nginx: built by gcc 4.4.3 (Ubuntu 4.4.3-4ubuntu5)
> nginx: TLS SNI support enabled
> nginx: configure arguments: --with-http_ssl_module
> --add-module=/gnosek-nginx-upstream-fair-2131c73
> --with-http_stub_status_module
> --add-module=/cep21-healthcheck_nginx_upstreams-b33a846
> --prefix=/usr/local/nginx-1.0.6 --with-debug
>
> used:
> peckhardt@test-nginx:~/nginx-1.0.6$patch -p1 <
> /cep21-healthcheck_nginx_upstreams-5fa4bff/nginx.patch
>
> hope someone would help me.
>
> greetings
>
> _______________________________________________
> nginx mailing list
> nginx@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx
>
_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx