Re: Haproxy running on 100% CPU and slow downloads

2016-05-09 Thread Sachin Shetty
We deployed the latest and we saw throughput still dropped around peak hours a bit, then we swithed to nbproc 4 which is holding up ok. Note that 4 Cpus was not sufficient earlier, so I believe the latest version is scaling better. Thanks Lukas and Willy. On 4/29/16, 11:09 AM, "Willy Tarreau"

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Willy Tarreau
Hi Maxime, On Tue, May 10, 2016 at 12:07:15AM +0200, Maxime de Roucy wrote: > > > I'm not sure to like this feature in its current implementation. > > > I fear it will also create some new issues depending on how people > > > will use it. > > Indeed it should be use with care. > But for me it's

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Willy Tarreau
On Tue, May 10, 2016 at 12:54:40AM +0200, maxime de Roucy wrote: > I forgot to free the memory allocated at 'filename = calloc' (why valgrind > didn't warn...). Forget this patch. I will send another one tomorow. Yes I noticed, and there's this one as well : > > +

update LED Floodlight/T5T8 Tube/LED Srtip price list 170lm/w

2016-05-09 Thread Karen-.yoshine lighting
If you do not wish to receive this message, please click here to unsubscribe!Dear Purchaseing manager,Have good day Very glad to tell you that:Our T5 T8 LED Tube Lumen Efficacy can be : 170LM/W  PF>0.9. You can spend only a little bit more time to know us in detail, you won't disappointed about 

SUPERLIGHTINGS @ Guangzhou lighting fair June 2016 (AD)

2016-05-09 Thread superlightings vincent

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread maxime de Roucy
I forgot to free the memory allocated at 'filename = calloc' (why valgrind didn't warn...). Forget this patch. I will send another one tomorow. Sorry Le 9 mai 2016 11:28 PM, "Maxime de Roucy" a écrit : > If -f argument is a directory add all the files (and only files)

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Pavlos Parissis
filter himself the files list ; it > can't with my patch. > >>> For example, I know lots of sysadmin who have the (bad) habit to >>> make backup of the configuration files in the same directory, >>> without cleaning it up. We may see some directories like this : >>> >

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Maxime de Roucy
Hi Willy, > > Instead I create Alert_exit which embedded the exit call inside > > Alert. > > I just sent the patchs. > > > > Is this solution good for you ? > > To be honnest, no, I really don't like it. There are very few such > locations that benefit from it and 2/3 of them even had the code

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Maxime de Roucy
(bad) habit to > > make backup of the configuration files in the same directory, > > without cleaning it up. We may see some directories like this : > > > > service1.cfg > > service2.cfg > > service2.cfg~ > > service3.cfg.20160509 > > service4.cfg-

[PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Maxime de Roucy
If -f argument is a directory add all the files (and only files) it containes to the config files list. These files are added in lexical order (man alphasort). The -f order is still respected: $ tree rootdir rootdir ├── root1 ├── root2 ├── root3 ├──

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Cyril Bonté
service2.cfg~ service3.cfg.20160509 service4.cfg-19980101 service5.cfg service5.cfg.old service6.cfg.disabled ...and so on. When several sysadmins share the same haproxy instance, it can quickly become annoying. Other use cases I immediately see : - some configurations provide the crt files

Unsubscribe

2016-05-09 Thread Errol Neal
Unsubscribe

Re: Stale UNIX sockets after reload

2016-05-09 Thread Pavlos Parissis
On 09/05/2016 02:26 μμ, Christian Ruppert wrote: > Hi, > > it seems that HAProxy does not remove the UNIX sockets after reloading > (also restarting?) even though they have been removed from the > configuration and thus are stale afterwards. > At least 1.6.4 seems to be affected. Can anybody else

Stale UNIX sockets after reload

2016-05-09 Thread Christian Ruppert
Hi, it seems that HAProxy does not remove the UNIX sockets after reloading (also restarting?) even though they have been removed from the configuration and thus are stale afterwards. At least 1.6.4 seems to be affected. Can anybody else confirm that? It's a multi-process setup in this case

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Willy Tarreau
On Mon, May 09, 2016 at 01:30:51PM +0200, Willy Tarreau wrote: > Hello Maxime, > > On Mon, May 09, 2016 at 11:17:25AM +0200, Maxime de Roucy wrote: > > > I have another comment here, please try to factor the error messages > > > using a goto, this block appears at least 3 times : > > > > I tried

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Willy Tarreau
Hello Maxime, On Mon, May 09, 2016 at 11:17:25AM +0200, Maxime de Roucy wrote: > > I have another comment here, please try to factor the error messages > > using a goto, this block appears at least 3 times : > > I tried to use goto but there is always a slight difference in the > Alert(...)

Re: AWS ELB with SSL backend adds proxy protocol inside SSL stream

2016-05-09 Thread Hector Rivas Gandara
On 5 May 2016 at 23:27, Igor Cicimov wrote: > > > On 5 May 2016 10:39 pm, "Hector Rivas Gandara" > wrote: > > > https://jve.linuxwall.info/ressources/taf/haproxy-aws/ > > Thank you for your answer, but this

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Maxime de Roucy
Hello, Thanks for your remarks ! > I think this is a nice addition, it completes well the ability to > load an arbitrary file list. Good to hear that :) > However I cannot merge it as is, there is a huge buffer overflow > … Ok, I see. I will try to patch that this evening. > I have another

[PATCH 2/2] MINOR: log: use Alert_exit in src/haproxy.c

2016-05-09 Thread Maxime de Roucy
This patch replace Alert(...); exit(1); with Alert(1, ...) when it's possible. --- src/haproxy.c | 176 +- 1 file changed, 87 insertions(+), 89 deletions(-) diff --git a/src/haproxy.c b/src/haproxy.c index 0c223e5..c1acea6 100644 ---

[PATCH 1/2] MINOR: log: Alert_exit function creation

2016-05-09 Thread Maxime de Roucy
Alert is often followed by exit. Alert_exit embedded the exit call. --- include/proto/log.h | 9 + src/log.c | 31 ++- 2 files changed, 35 insertions(+), 5 deletions(-) diff --git a/include/proto/log.h b/include/proto/log.h index e606a3c..7afa2a0

Re: [PATCH] MEDIUM: init: allow directory as argument of -f

2016-05-09 Thread Willy Tarreau
Hi, On Sun, May 08, 2016 at 11:41:17AM +0200, Maxime de Roucy wrote: > If -f argument is a directory add all the files (and only files) it > containes to the config files list. I think this is a nice addition, it completes well the ability to load an arbitrary file list. However I cannot merge