Hi,

Please try:

https://github.com/liseen/healthcheck_nginx_upstreams/blob/master/healthcheck.patch

patch -p1 < healthcheck.patch
../configure ....

if you use healthcheck with upstream hash, please compile with branch
support_http_healthchecks of cep21's fork

https://github.com/cep21/nginx_upstream_hash/tree/support_http_healthchecks

liseen


On Sat, Oct 1, 2011 at 6:06 AM, liseen <liseen.wan@gmail.com> wrote:

> 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