Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Simon, On Mon, Mar 13, 2017 at 02:36:28PM +0100, Simon Horman wrote: > Hi Willy, > > this patchset seems to have stalled. > I'd like to find a way to revive it. Good news now, I've just merged it! It looked good enough. I've run a few tests on it and validated that I didn't observerve any

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Simon, On Mon, Mar 13, 2017 at 02:36:28PM +0100, Simon Horman wrote: > Hi Willy, > > this patchset seems to have stalled. > I'd like to find a way to revive it. Just in time :-) I was about to ask if you had any news, but apparently I was the one stalling here. > > > Thanks for this. I'm

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Willy, this patchset seems to have stalled. I'd like to find a way to revive it. On Sun, Jan 08, 2017 at 11:52:46AM +0100, Simon Horman wrote: > Hi Willy, > > On Sun, Jan 08, 2017 at 07:37:24AM +0100, Willy Tarreau wrote: > > Hi Simon, > > > > On Wed, Jan 04, 2017 at 09:37:24AM +0100, Simon

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

On Thu, Jan 12, 2017 at 03:14:27PM +, Scott McKeown wrote: > Doh, that would do it. > Sorry wrong git branch. > > # haproxy -v > HA-Proxy version 1.8-dev0 2016/11/25 > Copyright 2000-2016 Willy Tarreau > > # echo "show info json" | socat /tmp/haproxy.stat stdio | python

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Doh, that would do it. Sorry wrong git branch. # haproxy -v HA-Proxy version 1.8-dev0 2016/11/25 Copyright 2000-2016 Willy Tarreau # echo "show info json" | socat /tmp/haproxy.stat stdio | python -m json.tool [ { "field": { "name": "Name",

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

On Thu, Jan 12, 2017 at 01:27:37PM +, Scott McKeown wrote: > Hi Simon, > > Output below: ... Thanks > On 12 January 2017 at 13:23, Simon Horman wrote: > > > Hi Scott, > > > > could you send the output of the following? > > > > echo "show info json" | socat

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Simon, Output below: # echo "show info json" | socat /tmp/haproxy.stat stdio Name: HAProxy Version: 1.6-dev1 Release_date: 2015/03/11 Nbproc: 1 Process_num: 1 Pid: 1611 Uptime: 0d 0h14m04s Uptime_sec: 844 Memmax_MB: 0 Ulimit-n: 4037 Maxsock: 4037 Maxconn: 2000 Hard_maxconn: 2000 CurrConns: 1

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Scott, could you send the output of the following? echo "show info json" | socat /tmp/haproxy.stat stdio On Thu, Jan 12, 2017 at 01:18:54PM +, Scott McKeown wrote: > Hi Guys, > Sorry for the delay I got tied up with some other issues yesterday but I've > just finished with Simons git

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Sorry forgot to show the /tmp/ folder # ls -latrh /tmp/ total 16K -rw---. 1 root root0 Jan 9 11:00 yum.log drwxrwxrwt. 2 root root 4.0K Jan 9 11:21 .ICE-unix dr-xr-xr-x. 22 root root 4.0K Jan 9 11:21 .. srw---. 1 root root0 Jan 9 14:39 haproxy.stat -rw-r--r--. 1 root root

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Guys, Sorry for the delay I got tied up with some other issues yesterday but I've just finished with Simons git repo pull. Simon are you sure this is correct as I thought this was for a 1.8-dev build which could be why I had problems on Tuesday. # haproxy -v HA-Proxy version 1.6-dev1

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

No problem I'll have another look tomorrow morning and I'll let you all know how I get on. On 9 January 2017 at 15:18, Simon Horman wrote: > Hi Scott, > > thanks for testing. > > For reference the code I am using is here: > > https://github.com/horms/haproxy.git show-json > >

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Scott, On Mon, Jan 09, 2017 at 03:10:10PM +, Scott McKeown wrote: > Hi Everyone, > > Simon I've been playing with this patch today and I'm sorry to say that > adding it to a 1.8 dev build as of today does not look to work. > > I did have to manually update a few of the patch chunks so

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Scott, thanks for testing. For reference the code I am using is here: https://github.com/horms/haproxy.git show-json And my minimal config file is as follows. Would it be possible for you to share you config with me (privately) ? global daemon stats socket /tmp/haproxy.stat

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Everyone, Simon I've been playing with this patch today and I'm sorry to say that adding it to a 1.8 dev build as of today does not look to work. I did have to manually update a few of the patch chunks so I'm willing to admit that it could be me that broke it and I will be having another go

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Willy, On Sun, Jan 08, 2017 at 07:37:24AM +0100, Willy Tarreau wrote: > Hi Simon, > > On Wed, Jan 04, 2017 at 09:37:24AM +0100, Simon Horman wrote: > > Hi, > > > > this short series is an RFC implementation of adding JSON format > > output to show (info|stat). It also adds a new show schema

Re: [PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi Simon, On Wed, Jan 04, 2017 at 09:37:24AM +0100, Simon Horman wrote: > Hi, > > this short series is an RFC implementation of adding JSON format > output to show (info|stat). It also adds a new show schema json > stats command to allow retreival of the schema which describes > the JSON output

[PATCH 0/2] MEDIUM: stats: Add JSON output option to show (info|stat)

Hi, this short series is an RFC implementation of adding JSON format output to show (info|stat). It also adds a new show schema json stats command to allow retreival of the schema which describes the JSON output of show (info|stat). Some areas for possible discussion: * Use of STAT_STARTED in