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. At least adding the AS number should be done. -- :wq Claudio
