Re: HAProxy Healthcheck issue using Virtual hostname

Hi I have an app deployed in Pivotal Cloudfoundry (PCF) and to route traffic to an app in PCF, we have to use application route name (virtual hostname). We have PCF in two different datacenters and I need to load balance the traffic to these DCs , but I'm having the challenge in checking the

Re: 1.9dev LUA shows partial results from print_r(core.get_info()) after adding headers ?

Hi Thierry, Op 3-5-2018 om 8:59 schreef Thierry Fournier: he bug. I even installed a FreeBSD:-) I add Willy in copy, maybe he will reproduce it. Thierry The 'trick' is probably sending as few requests as possible through a 'high latency' vpn (17ms for a ping from client to haproxy

Re: [PATCH] BUG/MINOR: lua: Socket.send causing 'close' needs 1 arguments.

> On 3 May 2018, at 19:54, Sadasiva Gujjarlapudi > wrote: > > `check_args` checked for the "exact number of" arguments available on the Lua > stack as > given in the second param(nb). You’re right ! Good catch. Can you provide the patch with this lot of explanation

Re: [PATCH] BUG/MINOR, lua/sockets, make lua tasks that are waiting for io suspend until woken up by the a corresponding event.

On Thu, May 03, 2018 at 08:37:09PM +0200, PiBa-NL wrote: > Hi Tim, Willy, > > Apparently even a simple copy/paste is to difficult for me to do right > sometimes really sorry about that.. :/ Bah don't worry, if you want to re-inflate your ego, just look in the logs for cleanups I've done just

Re: [PATCH] BUG/MINOR, lua/sockets, make lua tasks that are waiting for io suspend until woken up by the a corresponding event.

Hi Tim, Willy, Apparently even a simple copy/paste is to difficult for me to do right sometimes really sorry about that.. :/ Thanks for merging and explaining, ill try and do better next time :) Regards, PiBa-NL

Re: [PATCH] BUG/MINOR: lua: Socket.send causing 'close' needs 1 arguments.

`check_args` checked for the "exact number of" arguments available on the Lua stack as given in the second param(nb). So `close` was expecting exactly "one" argument on the Lua stack. But when `close` was called from xxx_write_yield(), Lua stack had 3 arguments. So close threw the exception with

Re: Considering adding support for TCP Zero Copy

On Thu, May 03, 2018 at 02:51:12PM +0200, Pavlos Parissis wrote: > On 03/05/2018 02:45 uu, Olivier Houchard wrote: > > Hi Pavlos, > > > > On Thu, May 03, 2018 at 12:45:42PM +0200, Pavlos Parissis wrote: > >> Hi, > >> > >> Linux kernel version 4.14 adds support for zero-copy from user memory to >

Re: Building on AIX 7.2

On Sat, Apr 28, 2018 at 01:08:17PM +, Donald MacKerracher wrote: > This is just for a quick sanity check. I'm not seeing much sign of any > mention of haproxy on AIX beyond 5.3. Before I involve myself heavily in it, > has anyone attempted a build of haproxy on AIX 7.2 or am I highly likely on

Re: [PATCH] MINOR: add get_maxconn and set_maxconn to LUA Server class.

applied, thanks! Willy

Re: [PATCH] MINOR: Add server name & puid to LUA Server class.

On Thu, May 03, 2018 at 03:35:41PM +0200, Thierry Fournier wrote: > Ok, I understand ... I hesitate. I think that is better to keep > the original type, but the consistency is a good reason to convert > to string. > > I doubt that integer operations (add, sub, ...) will be performed on > this id.

Re: [PATCH 1/1] DOC/MINOR: clean up LUA documentation re: servers & array/table.

On Thu, May 03, 2018 at 09:20:15AM +0200, Thierry Fournier wrote: > Hi Patrick, > > Thanks for cleaning the Lua doc. > > Willy, could you apply these patches ? Now done, thanks! Willy

Re: http-response set-header is unreliable

On Thu, May 03, 2018 at 06:00:45PM +0200, Tim Düsterhus wrote: > > What you have above looks like stderr. The rest are logs. They are for > > very different usages, stderr is there to inform you that something went > > wrong during a reload operation (that systemd happily hides so that you > >

Re: http-response set-header is unreliable

Willy, Am 03.05.2018 um 17:37 schrieb Willy Tarreau: >> [1] I'd love to have a proper integration with systemd-journald to have >> all my logs in one place. It's pretty annoying, because some things >> ("Proxy bk_*** started"; [WARNING] 121/202559 (11635) : Reexecuting >> Master process) go to

Re: http-response set-header is unreliable

Hi Tim, On Thu, May 03, 2018 at 03:34:01PM +0200, Tim Düsterhus wrote: > >> Especially since the issue happens randomly: Sometimes the additional > >> headers fit by chance. Sometimes they don't. I would start by > >> investigating the connection to the backend services, not investigating > >>

Re: [PATCH] MINOR: Add server name & puid to LUA Server class.

Thierry, Am 03.05.2018 um 15:35 schrieb Thierry Fournier: > Ok, I understand ... I hesitate. I think that is better to keep > the original type, but the consistency is a good reason to convert > to string. > > I doubt that integer operations (add, sub, ...) will be performed on > this id. So

Re: [PATCH] MINOR: Add server name & puid to LUA Server class.

> On 3 May 2018, at 14:27, Patrick Hemmer wrote: > > > > On 2018/5/3 02:52, Thierry Fournier wrote: >>> On 2 May 2018, at 16:49, Willy Tarreau >>> wrote: >>> >>> Hi Thierry, >>> >>> when you have a moment, could you please give a quick look at these

Re: http-response set-header is unreliable

Willy, Am 03.05.2018 um 05:23 schrieb Willy Tarreau: >> To me a message like: "Unable to add-header Content-Security-Policy to >> response. Possibly the amount of headers exceeds tune.maxrewrite." would >> have been more helpful than random 502 without any further information. > > We could

Re: LUA Converters should be global

Hi Patrick, You’re right. This question was already ask. I can’t move the converters in the global context because some converters linked with a session. for example: - capture-req - capture-res - set-var - unset-var Maybe we can add a flag to each converter to known if they use a session

Re: [PATCH] BUG/MINOR: lua: Socket.send causing 'close' needs 1 arguments.

Hi, I’m not sure about your patch because the values in the stack are kept. I need to be focus to read it, and unfortunately I’m very busy. I try to check this morning, during the french strikes... Thierry > On 25 Apr 2018, at 20:26, Sadasiva Gujjarlapudi > wrote: >

Re: 1.9dev LUA shows partial results from print_r(core.get_info()) after adding headers ?

Hi, I can’t reproduce the bug. I even installed a FreeBSD :-) I add Willy in copy, maybe he will reproduce it. Thierry > On 28 Apr 2018, at 01:36, PiBa-NL wrote: > > Hi Thierry, > > Op 27-4-2018 om 1:54 schreef PiBa-NL: >> Hi Thierry, >> >> Op 26-4-2018 om 12:25

Re: Considering adding support for TCP Zero Copy

On 03/05/2018 02:45 μμ, Olivier Houchard wrote: > Hi Pavlos, > > On Thu, May 03, 2018 at 12:45:42PM +0200, Pavlos Parissis wrote: >> Hi, >> >> Linux kernel version 4.14 adds support for zero-copy from user memory to TCP >> sockets by setting >> MSG_ZEROCOPY flag. This is for the sending side of

Re: Considering adding support for TCP Zero Copy

Hi Pavlos, On Thu, May 03, 2018 at 12:45:42PM +0200, Pavlos Parissis wrote: > Hi, > > Linux kernel version 4.14 adds support for zero-copy from user memory to TCP > sockets by setting > MSG_ZEROCOPY flag. This is for the sending side of the socket, for the > receiving side of the socket > we

Re: [PATCH] BUG/MINOR, lua/sockets, make lua tasks that are waiting for io suspend until woken up by the a corresponding event.

Pieter, Am 03.05.2018 um 00:58 schrieb PiBa-NL: > *snip* As for line > lengths, it didn't even cross my mind to look at that. (i also didn't > lookup the contributing either sorry, though it doesn't seem to specify > specific line lengths.?.) > Indeed they are not specified for haproxy, but the

Re: [PATCH] MINOR: Add server name & puid to LUA Server class.

On 2018/5/3 02:52, Thierry Fournier wrote: >> On 2 May 2018, at 16:49, Willy Tarreau wrote: >> >> Hi Thierry, >> >> when you have a moment, could you please give a quick look at these >> patches from Patrick so that I know if I can merge them or not ? There >> are 2 other ones on

Re: [PATCH] MINOR: Add server name & puid to LUA Server class.

> On 2 May 2018, at 16:49, Willy Tarreau wrote: > > Hi Thierry, > > when you have a moment, could you please give a quick look at these > patches from Patrick so that I know if I can merge them or not ? There > are 2 other ones on the list. Hi Willy and Patrick, I check it. I

Re: [PATCH] BUG/MINOR: lua: Socket.send causing 'close' needs 1 arguments.

This function checks that one argument are present in the stack, at the bottom of the stack (position 1). MAY_LJMP(check_args(L, 1, "close")); The stack is kept between calls, so the position 1 will contains anything. The content of the position 1 in the stack is a struct associated with the

Re: [PATCH 1/1] DOC/MINOR: clean up LUA documentation re: servers & array/table.

Hi Patrick, Thanks for cleaning the Lua doc. Willy, could you apply these patches ? Theses patch could be backported from 1.8 to 1.6, but it is not so important. Thierry > On 2 May 2018, at 03:30, Patrick Hemmer wrote: > > > * A few typos > * Fix definitions of

Considering adding support for TCP Zero Copy

Hi, Linux kernel version 4.14 adds support for zero-copy from user memory to TCP sockets by setting MSG_ZEROCOPY flag. This is for the sending side of the socket, for the receiving side of the socket we need to wait for kernel version 4.18. Will you consider enabling this on HAProxy? More

Sticky sessions

Hello guys Sorry that I'm distracting you but I was struggling with an error for 1 week and couldnt solve it by myself. Maybe you will have time to check my config to help a bit I've even made a topic in stackoverflow https://stackoverflow.com/questions/50149747/haproxy-sticky-sessions