Re: [patch] bgpctl more info in terse format

2016-05-06 Thread Denis Fondras
On Mon, Apr 18, 2016 at 08:09:50PM +0200, Claudio Jeker wrote: > The idea of terse is that you don't need to parse. So in a way I agree > with the diff. What I don't like is the inclusion of the number of > prefixes. That count requires a roundtrip to the RDE to find and sometimes > this takes a wh

Re: [patch] bgpctl more info in terse format

2016-04-19 Thread Denis Fondras
On Mon, Apr 18, 2016 at 08:09:50PM +0200, Claudio Jeker wrote: > The idea of terse is that you don't need to parse. So in a way I agree > with the diff. What I don't like is the inclusion of the number of > prefixes. That count requires a roundtrip to the RDE to find and sometimes > this takes a wh

Re: [patch] bgpctl more info in terse format

2016-04-18 Thread Claudio Jeker
On Mon, Apr 18, 2016 at 01:50:09AM +0200, Sebastian Benoit wrote: > If you do that, then you can also just parse the output of "bgpctl show > sum", no? > The idea of terse is that you don't need to parse. So in a way I agree with the diff. What I don't like is the inclusion of the number of prefi

Re: [patch] bgpctl more info in terse format

2016-04-18 Thread Sebastian Benoit
If you do that, then you can also just parse the output of "bgpctl show sum", no? /Benno Denis Fondras(open...@ledeuns.net) on 2016.04.17 18:09:18 +0200: > Hello, > > When monitoring my bgpd, I need to check the session duration and the number > of > prefixes. Here is a patch that add these inf

Re: [patch] bgpctl more info in terse format

2016-04-17 Thread Denis Fondras
> If you do that, then you can also just parse the output of "bgpctl show > sum", no? > Of course but I would have to parse day/hour/minute/second. It is simpler if bgpd can give me the value straight. Denis

[patch] bgpctl more info in terse format

2016-04-17 Thread Denis Fondras
Hello, When monitoring my bgpd, I need to check the session duration and the number of prefixes. Here is a patch that add these informations to "bgpctl show sum terse" Before : # bgpctl show sum terse 10.20.30.254 65003 Established After : # bgpctl show sum terse 10.20.30.254 65003 Established 1