Re: [PATCH 0/3] prometheus: add listen stats

2021-02-15 Thread Christopher Faulet

Le 14/02/2021 à 23:22, William Dauchy a écrit :

Hello Christopher,

I know I'm a bit late regarding the merge window; this is however the
logical followup of my prometheus work, now adding listen stats.
patch 2/3 is a proposition but I'm ok to revisit.

William Dauchy (3):
   MEDIUM: stats: allow to select one field in `stats_fill_li_stats`
   MINOR: stats: add helper to get status string
   MEDIUM: contrib/prometheus-exporter: add listen stats

  contrib/prometheus-exporter/README|  24 +-
  .../prometheus-exporter/service-prometheus.c  | 274 --
  include/haproxy/listener-t.h  |   9 +
  include/haproxy/listener.h|   3 +
  include/haproxy/stats.h   |   2 +-
  reg-tests/contrib/prometheus.vtc  |   4 +
  src/hlua_fcn.c|   3 +-
  src/listener.c|  18 ++
  src/stats.c   | 166 +++
  9 files changed, 365 insertions(+), 138 deletions(-)



Thanks William, now merged, including the changes discussed offlist.

--
Christopher Faulet



[PATCH 0/3] prometheus: add listen stats

2021-02-14 Thread William Dauchy
Hello Christopher,

I know I'm a bit late regarding the merge window; this is however the
logical followup of my prometheus work, now adding listen stats.
patch 2/3 is a proposition but I'm ok to revisit.

William Dauchy (3):
  MEDIUM: stats: allow to select one field in `stats_fill_li_stats`
  MINOR: stats: add helper to get status string
  MEDIUM: contrib/prometheus-exporter: add listen stats

 contrib/prometheus-exporter/README|  24 +-
 .../prometheus-exporter/service-prometheus.c  | 274 --
 include/haproxy/listener-t.h  |   9 +
 include/haproxy/listener.h|   3 +
 include/haproxy/stats.h   |   2 +-
 reg-tests/contrib/prometheus.vtc  |   4 +
 src/hlua_fcn.c|   3 +-
 src/listener.c|  18 ++
 src/stats.c   | 166 +++
 9 files changed, 365 insertions(+), 138 deletions(-)

-- 
2.30.0