Le 12/6/22 à 15:13, Cedric Paillet a écrit :
Sorry, indeed I mentioned the enum ID and not the promex name.
I proposed to keep "haproxy_bakcned_agg_server_check_status"
altering its description to announce it is deprecated
. And then to add "haproxy_backend_agg_check_status"
aggregate
>Sorry, indeed I mentioned the enum ID and not the promex name.
>I proposed to keep "haproxy_bakcned_agg_server_check_status"
> altering its description to announce it is deprecated
>. And then to add "haproxy_backend_agg_check_status"
>aggregate haproxy_server_check_status) and
On Tue, Dec 6, 2022 at 3:27 AM Christopher Faulet wrote:
> IMHO, we should keep the current metric and its description should be updated
> to
> mention it is deprecated. This way, we will be able to remove it in the 2.9 or
> 3.0. This means we should find new names for the aggregated servers
Le 12/6/22 à 12:26, Cedric Paillet a écrit :
Sorry for the delay. We were busy because of the 2.7.0 and then I forgot your
patches :) And it was a bit late to add it into the 2.7.0.
No problem.
Because the metric name is indeed badly chosen but we can't break the
compatibility, especially
> Sorry for the delay. We were busy because of the 2.7.0 and then I forgot your
> patches :) And it was a bit late to add it into the 2.7.0.
No problem.
> Because the metric name is indeed badly chosen but we can't break the
> compatibility, especially in a LTS version.
Ok, I understand that.
Le 11/28/22 à 08:44, Cedric Paillet a écrit :
As described in github issue #1312, the first intention of patch 42d7c402d
was to aggregate haproxy_server_check_status. But we aggregated
haproxy_server_status instead.
To fix that, rename haproxy_backend_agg_server_check_status to
Envoyé : mardi 29 novembre 2022 12:39
À : Cedric Paillet
Cc : haproxy@formilux.org; Christopher Faulet
Objet : Re: [PATCH 0/2] BUG/MINOR: promex: fix
haproxy_backend_agg_server_check_status
On Tue, Nov 29, 2022 at 11:52 AM William Dauchy wrote:
> On Mon, Nov 28, 2022 at 8:46 AM Cedric Pail
On Tue, Nov 29, 2022 at 11:52 AM William Dauchy wrote:
> On Mon, Nov 28, 2022 at 8:46 AM Cedric Paillet wrote:
> > As described in github issue #1312, the first intention of patch 42d7c402d
> > was to aggregate haproxy_server_check_status. But we aggregated
> > haproxy_server_status instead.
> >
Hello Cedric,
On Mon, Nov 28, 2022 at 8:46 AM Cedric Paillet wrote:
> As described in github issue #1312, the first intention of patch 42d7c402d
> was to aggregate haproxy_server_check_status. But we aggregated
> haproxy_server_status instead.
> To fix that, rename
9 matches
Mail list logo
