Hello Ciara,

On Thu, Oct 20, 2022 at 11:54 AM Power, Ciara <[email protected]> wrote:
> This is a nice addition to the library.
>
> > -----Original Message-----
> > From: David Marchand <[email protected]>
> > Sent: Wednesday 19 October 2022 08:37
> > To: [email protected]
> > Cc: Richardson, Bruce <[email protected]>;
> > [email protected]; Power, Ciara <[email protected]>
> > Subject: [PATCH] telemetry: support boolean type
> >
> > Add the boolean type RTE_TEL_BOOL_VAL for values in arrays and dicts.
> >
> > Signed-off-by: David Marchand <[email protected]>
> > ---
> >  app/test/test_telemetry_data.c | 88 +++++++++++++++++++++++++++++++++-
> >  lib/telemetry/rte_telemetry.h  | 36 ++++++++++++++
> >  lib/telemetry/telemetry.c      | 24 +++++++++-
> >  lib/telemetry/telemetry_data.c | 44 +++++++++++++++--
> > lib/telemetry/telemetry_data.h |  5 ++  lib/telemetry/telemetry_json.h | 34
> <snip>
>
> Acked-by: Ciara Power <[email protected]>

Thank you.

Fyi, I'll respin this patch to fix Bruce comments.
I will send it as preparation for the trace telemetry additions.


-- 
David Marchand

Reply via email to