Claudio Jeker(cje...@diehard.n-r-g.com) on 2019.09.24 17:01:21 +0200: > On Tue, Sep 24, 2019 at 03:51:43PM +0100, Stuart Henderson wrote: > > On 2019/09/24 11:10, Claudio Jeker wrote: > > > On Tue, Sep 24, 2019 at 10:06:51AM +0100, Stuart Henderson wrote: > > > > On 2019/09/23 22:48, Stuart Henderson wrote: > > > > > "bgpctl XX nei" functions can now take "group XX" - when used as > > > > > "show nei group XX terse" you can't tell which entry relates to each > > > > > neighbour. > > > > > > > > > > OK to add the address to the end of the line where it's reasonably > > > > > out of the way of existing parsers? > > > > > > > > missing free, pointed out by benno. (not that bgpctl will stick around > > > > for long anyway :) > > > > > > This is fine with me. I wonder if other data e.g. the peer description or > > > peer AS number should be added as well. > > > > That might be useful, though as the peer description could contain > > spaces we'd want to be reasonably sure we have already included any > > other useful data first so that it can go right at the end of the line > > (so that parsing the output isn't too awkward). > > > > I was thinking the same maybe even put the name in "" to make it look more > like a string.
Yes, that makes it matchable again at least. > At least adding the AS number should be done. +1 > > -- > :wq Claudio >