WHY they are different when checking concurrent limit?

2015-11-09 Thread Zhou,Qingzhi
Hi guys: I’m reading the source code of version 1.6.2, in function listener_accept: /* Note: if we fail to allocate a connection because of configured * limits, we'll schedule a new attempt worst 1 second later in the * worst case. If we fail due to system limits or temporary resource *

Re: Debug mode not working?!

2015-11-09 Thread Willy Tarreau
On Mon, Nov 09, 2015 at 10:15:46PM +0100, Aleksandar Lazic wrote: > > ... > epoll_wait(3, {}, 200, 1000)= 0 > epoll_wait(3, {{EPOLLIN, {u32=5, u64=5}}}, 200, 1000) = 1 > accept4(5, {sa_family=AF_INET, sin_port=htons(52310), > sin_addr=inet_addr("127.0.0.1")}, [16],

Re: Debug mode not working?!

2015-11-09 Thread Aleksandar Lazic
Am 09-11-2015 11:34, schrieb Willy Tarreau: Hi Aleks, On Sun, Nov 08, 2015 at 04:24:29PM +0100, Aleksandar Lazic wrote: Hi. Today I have tried to debug haproxy as in the old days ;-), I was not able to see the communication on stderr. I'm sure I have something missed in the past on the

Re: Debug mode not working?!

2015-11-09 Thread Aleksandar Lazic
Am 09-11-2015 22:21, schrieb Willy Tarreau: On Mon, Nov 09, 2015 at 10:15:46PM +0100, Aleksandar Lazic wrote: ... epoll_wait(3, {}, 200, 1000)= 0 epoll_wait(3, {{EPOLLIN, {u32=5, u64=5}}}, 200, 1000) = 1 accept4(5, {sa_family=AF_INET, sin_port=htons(52310),

Re: Debug mode not working?!

2015-11-09 Thread Nenad Merdanovic
Hello Aleksandar, > Okay after removing accept-proxy from > > bind *:${HTTP_BIND_PORT} accept-proxy tfo > > It comes what expected. If you are using 'accept-proxy', HAproxy expects the payload to start with a PROXY protocol header. http://www.haproxy.org/download/1.6/doc/proxy-protocol.txt

Re: Debug mode not working?!

2015-11-09 Thread Aleksandar Lazic
Hi Nenad, Am 09-11-2015 22:52, schrieb Nenad Merdanovic: Hello Aleksandar, Okay after removing accept-proxy from bind *:${HTTP_BIND_PORT} accept-proxy tfo It comes what expected. If you are using 'accept-proxy', HAproxy expects the payload to start with a PROXY protocol header.

appsession replacement in 1.6

2015-11-09 Thread Sylvain Faivre
Hi, Sorry I'm late on this discussion, following this thread : https://marc.info/?l=haproxy=143345620219498=2 We are using appsession with HAproxy 1.5 like this : backend http appsession JSESSIONID len 24 timeout 1h request-learn We would like to be able to do the same thing with HAproxy

Re: MINOR: Makefile deviceatlas

2015-11-09 Thread Willy Tarreau
Thanks David, merged into 1.7 and 1.6. Willy

Re: WHY they are different when checking concurrent limit?

2015-11-09 Thread Willy Tarreau
Hi, On Mon, Nov 09, 2015 at 12:46:57PM +, Zhou,Qingzhi wrote: > if (unlikely(actconn >= global.maxconn) && !(l->options & LI_O_UNLIMITED)) { > limit_listener(l, _listener_queue); > task_schedule(global_listener_queue_task, tick_add(now_ms, 1000)); /* try > again in 1 second */ > return; > }

Re: [PATCH] MEDIUM: mailer: try sending a mail up to 3 times

2015-11-09 Thread Willy Tarreau
On Tue, Nov 10, 2015 at 09:52:21AM +0900, Simon Horman wrote: > I would slightly prefer if there was a more substantial comment in > process_email_alert() noting that retry occurs 3 times. Good point, I'll add this when merging. > But regardless: > Acked-by: Simon Horman

Re: WHY they are different when checking concurrent limit?

2015-11-09 Thread Zhou,Qingzhi
Hi: Thanks very much. But I think we can use listener_full instead of limit_listener if we want wake up the listener when there’s a connection closed. Like in the beginning of listener_accept: if (unlikely(l->nbconn >= l->maxconn)) { listener_full(l); return; } WHY not

Re:

2015-11-09 Thread Hoggins!
Le 09/11/2015 11:39, Willy Tarreau a écrit : > On Sat, Nov 07, 2015 at 09:55:39PM +0100, Baptiste wrote: >> Hi, >> >> This is an english mailing list! > And I just checked, he is *not* subscribed to the list! > > Willy > > Yup. I wouldn't be surprised if haproxy@formilux.org was used by bots as

Re: Debug mode not working?!

2015-11-09 Thread Willy Tarreau
Hi Aleks, On Sun, Nov 08, 2015 at 04:24:29PM +0100, Aleksandar Lazic wrote: > Hi. > > Today I have tried to debug haproxy as in the old days ;-), I was not > able to see the communication on stderr. > > I'm sure I have something missed in the past on the list to be able to > see the output.

Re:

2015-11-09 Thread Willy Tarreau
On Sat, Nov 07, 2015 at 09:55:39PM +0100, Baptiste wrote: > Hi, > > This is an english mailing list! And I just checked, he is *not* subscribed to the list! Willy

Re: [PATCH] MEDIUM: mailer: try sending a mail up to 3 times

2015-11-09 Thread Simon Horman
On Mon, Nov 09, 2015 at 11:11:53AM +0100, Willy Tarreau wrote: > Hi Pieter, > > > Hi Ben, Willy, Simon, > > > > Ben, thanks for the review. > > Hoping 'release pressure' has cleared for Willy i'm resending the > > patch now, with with your comments incorporated. > > > > CC, to Simon as

Re: appsession replacement in 1.6

2015-11-09 Thread Aleksandar Lazic
Hi Sylvain Faivre. Am 09-11-2015 17:31, schrieb Sylvain Faivre: Hi, Sorry I'm late on this discussion, following this thread : https://marc.info/?l=haproxy=143345620219498=2 We are using appsession with HAproxy 1.5 like this : Thanks ;-) backend http appsession JSESSIONID len 24

RE: HAProxy with multiple CRL's

2015-11-09 Thread Harvan, Michael P
Thank you for the information. The Root CA did indeed have a CRL. I added that to my combined CRL, in the order local CRL, external root CRL, external intermediate CRL and everything worked. Thanks again, Mike -Original Message- From: Toft Alex (HEALTH AND SOCIAL CARE INFORMATION

Re: [PATCH] DOC: lua-api/index.rst small example fixes, spelling correction.

2015-11-09 Thread Willy Tarreau
Hi Pieter, On Sun, Nov 08, 2015 at 04:44:18PM +0100, PiBa-NL wrote: > Hi List, Willy, > > Attached some small example fixes, spelling correction. > Hope its ok like this :). Applied, thanks! Willy

Re: [PATCH] MEDIUM: mailer: try sending a mail up to 3 times

2015-11-09 Thread Willy Tarreau
Hi Pieter, > Hi Ben, Willy, Simon, > > Ben, thanks for the review. > Hoping 'release pressure' has cleared for Willy i'm resending the > patch now, with with your comments incorporated. > > CC, to Simon as maintainer of mailers part so he can give approval (or > not). > > The original