Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-04-28 Thread Willy Tarreau
Hi Alex, On Fri, Apr 28, 2023 at 11:43:26AM +0200, Aleksandar Lazic wrote: > Attached the new patch. Thank you, it went OK on the CI so we don't even need to refine the list of targets for now. I've just merged it as-is. Many thanks for your fast update and for the reminder! Willy

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-04-28 Thread Aleksandar Lazic
: Add bc_rtt and bc_rttvar This Patch adds fetch samples for backends round trip time. --- doc/configuration.txt| 16 ++ reg-tests/sample_fetches/tcpinfo_rtt.vtc | 39 src/tcp_sample.c | 32 +++ 3 files

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-04-28 Thread Aleksandar Lazic
Hi Will. On 28.04.23 11:07, Willy Tarreau wrote: Hi Alex, On Fri, Apr 28, 2023 at 10:59:46AM +0200, Aleksandar Lazic wrote: Hi Willy. On 30.03.23 06:23, Willy Tarreau wrote: On Thu, Mar 30, 2023 at 06:16:34AM +0200, Willy Tarreau wrote: Hi Alex, On Wed, Mar 29, 2023 at 04:06:10PM +0200,

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-04-28 Thread Willy Tarreau
Hi Alex, On Fri, Apr 28, 2023 at 10:59:46AM +0200, Aleksandar Lazic wrote: > Hi Willy. > > On 30.03.23 06:23, Willy Tarreau wrote: > > On Thu, Mar 30, 2023 at 06:16:34AM +0200, Willy Tarreau wrote: > > > Hi Alex, > > > > > > On Wed, Mar 29, 2023 at 04:06:10PM +0200, Aleksandar Lazic wrote: > >

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-04-28 Thread Aleksandar Lazic
Hi Willy. On 30.03.23 06:23, Willy Tarreau wrote: On Thu, Mar 30, 2023 at 06:16:34AM +0200, Willy Tarreau wrote: Hi Alex, On Wed, Mar 29, 2023 at 04:06:10PM +0200, Aleksandar Lazic wrote: Ping? thanks for the ping, I missed it a few times when being busy with some painful bugs in the past.

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-03-29 Thread Willy Tarreau
On Thu, Mar 30, 2023 at 06:16:34AM +0200, Willy Tarreau wrote: > Hi Alex, > > On Wed, Mar 29, 2023 at 04:06:10PM +0200, Aleksandar Lazic wrote: > > Ping? > > thanks for the ping, I missed it a few times when being busy with some > painful bugs in the past. I've pushed it to a topic branch to

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-03-29 Thread Willy Tarreau
Hi Alex, On Wed, Mar 29, 2023 at 04:06:10PM +0200, Aleksandar Lazic wrote: > Ping? thanks for the ping, I missed it a few times when being busy with some painful bugs in the past. I've pushed it to a topic branch to verify what it does on the CI for non-linux OS; we might have to add a "feature

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-03-29 Thread Aleksandar Lazic
Ping? On 10.01.23 21:27, Aleksandar Lazic wrote: On 09.12.22 13:17, Aleksandar Lazic wrote: Hi. As I still think that the Balancing algorithm (Peak) EWMA ( https://github.com/haproxy/haproxy/issues/1570 ) could help to make a "better" decision to which server should the request be send,

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-02-16 Thread Aleksandar Lazic
Hi. Any chance to add this Patch? Regards Alex On 10.01.23 21:27, Aleksandar Lazic wrote: On 09.12.22 13:17, Aleksandar Lazic wrote: Hi. As I still think that the Balancing algorithm (Peak) EWMA ( https://github.com/haproxy/haproxy/issues/1570 ) could help to make a "better" decision to

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2023-01-10 Thread Aleksandar Lazic
0e65d7dbc Mon Sep 17 00:00:00 2001 From: Aleksandar Lazic Date: Fri, 9 Dec 2022 13:05:52 +0100 Subject: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar This Patch adds the fetch sample for backends round trip time. --- doc/configuration.txt| 16 ++ reg-tests/sample_fet

Re: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2022-12-14 Thread Aleksandar Lazic
Hi, Any feedback to that patch? On 09.12.22 13:17, Aleksandar Lazic wrote: Hi. As I still think that the Balancing algorithm (Peak) EWMA ( https://github.com/haproxy/haproxy/issues/1570 ) could help to make a "better" decision to which server should the request be send, here the beginning

[PATCH] MINOR: sample: Add bc_rtt and bc_rttvar

2022-12-09 Thread Aleksandar Lazic
bject: [PATCH] MINOR: sample: Add bc_rtt and bc_rttvar To be able to implement "Balancing algorithm (Peak) EWMA" is it necessary to know the round trip time to the backend. This Patch adds the fetch sample for the backend server. Part of GH https://github.com/haproxy/haproxy/issues/15