On Fri, Oct 14, 2022 at 5:31 AM Chengwen Feng <fengcheng...@huawei.com> wrote:
>
> Currently, the dpdk-telemetry.py show json in raw format, which is not
> good for human reading.
>
> E.g. The command '/ethdev/xstats,0' will output:
> {"/ethdev/xstats": {"rx_good_packets": 0, "tx_good_packets": 0,
> "rx_good_bytes": 0, "tx_good_bytes": 0, "rx_missed_errors": 0,
> "rx_errors": 0, "tx_errors": 0, "rx_mbuf_allocation_errors": 0,
> "rx_q0_packets": 0,...}}
>
> This patch supports json pretty print by adding extra indent=4
> parameter, so the same command will output:
> {
> "/ethdev/xstats": {
> "rx_good_packets": 0,
> "tx_good_packets": 0,
> "rx_good_bytes": 0,
> "tx_good_bytes": 0,
> "rx_missed_errors": 0,
> "rx_errors": 0,
> "tx_errors": 0,
> "rx_mbuf_allocation_errors": 0,
> "rx_q0_packets": 0,
> ...
> }
> }
>
> Signed-off-by: Chengwen Feng <fengcheng...@huawei.com>
It's indeed easier to read, but maybe 4 chars is too much.
2 chars seem enough to me.
In any case I like the idea:
Acked-by: David Marchand <david.march...@redhat.com>
--
David Marchand
