Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-05-09 Thread Tim Düsterhus
Ilya, On 5/8/21 9:19 PM, Илья Шипицин wrote: I've created "proof of concept" OT support for CI https://github.com/chipitsine/haproxy/runs/2535689580 Tim, are you ok with that approach ? If yes, I will polish things a bit. If the build time is reduced appropriately then I am fine with that.

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-05-08 Thread Илья Шипицин
I've created "proof of concept" OT support for CI https://github.com/chipitsine/haproxy/runs/2535689580 Tim, are you ok with that approach ? If yes, I will polish things a bit. currently "building OT and satellites" takes 2min 11sec. seem, I will add similar caching level as for SSL variants

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-08 Thread Илья Шипицин
чт, 8 апр. 2021 г. в 14:25, Willy Tarreau : > On Wed, Apr 07, 2021 at 05:26:24PM +0500, ??? wrote: > > we run "all features anebled" gcc and clang builds, for example > > BUG/MINOR: tools: fix parsing "us" unit for timers · > > haproxy/haproxy@a683805 (github.com) > > < >

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-08 Thread Willy Tarreau
On Wed, Apr 07, 2021 at 05:26:24PM +0500, ??? wrote: > we run "all features anebled" gcc and clang builds, for example > BUG/MINOR: tools: fix parsing "us" unit for timers · > haproxy/haproxy@a683805 (github.com) >

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-08 Thread Willy Tarreau
Hi Miroslav! On Wed, Apr 07, 2021 at 11:32:16AM +0200, Miroslav Zagorac wrote: > Hello, > > due to the modified function declaration, the opentracing filter can no > longer be compiled. > > In commit 9533a7038 new parameters have been added to the declaration > of function parse_logsrv().

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-07 Thread Илья Шипицин
On Wed, Apr 7, 2021, 2:16 PM Miroslav Zagorac wrote: > On 04/07/2021 12:13 PM, Илья Шипицин wrote: > > do you consider adding opentracing to "github actions" CI ? > > > > Hello Илья, > > I don't know how to add it because I never used it. The filter uses > specific libraries that are not part

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-07 Thread Miroslav Zagorac
On 04/07/2021 12:13 PM, Илья Шипицин wrote: do you consider adding opentracing to "github actions" CI ? Hello Илья, I don't know how to add it because I never used it. The filter uses specific libraries that are not part of the system but need to be installed and/or compiled

Re: [PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-07 Thread Илья Шипицин
do you consider adding opentracing to "github actions" CI ? ср, 7 апр. 2021 г. в 14:35, Miroslav Zagorac : > Hello, > > due to the modified function declaration, the opentracing filter can no > longer be compiled. > > In commit 9533a7038 new parameters have been added to the declaration > of

[PATCH] MINOR: opentracing: register config file and line number on log servers

2021-04-07 Thread Miroslav Zagorac
. -- Zaga What can change the nature of a man? >From baae10ef8a2eb487328664a3ff04475cd5d76179 Mon Sep 17 00:00:00 2001 From: Miroslav Zagorac Date: Wed, 7 Apr 2021 11:14:23 +0200 Subject: [PATCH] MINOR: opentracing: register config file and line number on log servers In commit 9533a7038