mbox series

[iproute2-next,0/3] bond, bridge: add xstats json support

Message ID 20190312164128.22536-1-nikolay@cumulusnetworks.com
Headers show
Series bond, bridge: add xstats json support | expand

Message

Nikolay Aleksandrov March 12, 2019, 4:41 p.m. UTC 
Hi,
This set adds json output support to the xstats API (patch 01) and then
adds json support to the bridge xstats output (patch 02) and adds xstats
output support (both plain text and json) for the bonding (patch 03).
It doesn't change the bridge's plain text output, but it fixes an
inconsistency that could happen if new bridge xstats attributes were
added (print the interface name once for each group of xstats attrs).

Thanks,
 Nik


Nikolay Aleksandrov (3):
  ip: xstats: add json output support
  ip: bridge: add xstats json support
  ip: bond: add xstats support

 ip/ip_common.h         |   3 +
 ip/iplink_bond.c       | 167 +++++++++++++++++++++++++++++++++++++++-
 ip/iplink_bond_slave.c |   2 +
 ip/iplink_bridge.c     | 169 +++++++++++++++++++++++++----------------
 ip/iplink_xstats.c     |   3 +
 5 files changed, 276 insertions(+), 68 deletions(-)

Comments

David Ahern March 15, 2019, 9:03 p.m. UTC | #1 
On 3/12/19 10:41 AM, Nikolay Aleksandrov wrote:
> Hi,
> This set adds json output support to the xstats API (patch 01) and then
> adds json support to the bridge xstats output (patch 02) and adds xstats
> output support (both plain text and json) for the bonding (patch 03).
> It doesn't change the bridge's plain text output, but it fixes an
> inconsistency that could happen if new bridge xstats attributes were
> added (print the interface name once for each group of xstats attrs).
> 

applied to iproute2-next. Thanks