Le 12/8/22 à 10:16, 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:
- Deprecated haproxy_backend_agg_server_check_status. (Modify the metric 
description)
- create new haproxy_backend_agg_server_status metric, aggregation of 
haproxy_backend_server_status. (to replace misnamed 
haproxy_backend_agg_server_check_status)
- create new haproxy_backend_agg_check_status metric, aggregation of 
haproxy_backend_server_check_status.

Cedric Paillet (2):
   BUG/MINOR: promex: create haproxy_backend_agg_server_status
   MINOR: promex: introduce haproxy_backend_agg_check_status

  addons/promex/service-prometheus.c | 30 +++++++++++++++++++++++++++++-
  include/haproxy/stats-t.h          |  2 ++
  src/stats.c                        | 10 ++++++++--
  3 files changed, 39 insertions(+), 3 deletions(-)


Thanks, both patches were merged ! I mentioned it could be backported as far as 
2.4.

--
Christopher Faulet


Reply via email to