RE: [PATCH v2 00/13] telemetry JSON escaping and other enhancements

2022-08-23 Thread Power, Ciara
> -Original Message- > From: Bruce Richardson > Sent: Monday 25 July 2022 17:35 > To: dev@dpdk.org > Cc: Richardson, Bruce > Subject: [PATCH v2 00/13] telemetry JSON escaping and other > enhancements > > This patchset contains fixes for the problem of h

RE: [PATCH v2 00/13] telemetry JSON escaping and other enhancements

2022-07-27 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Wednesday, 27 July 2022 11.13 > > On Wed, Jul 27, 2022 at 09:51:04AM +0800, fengchengwen wrote: > > Hi Bruce, > > > > I think escape the string at begin (following function) seem more > simple: > > rte_tel_data_string > >

Re: [PATCH v2 00/13] telemetry JSON escaping and other enhancements

2022-07-27 Thread Bruce Richardson
On Wed, Jul 27, 2022 at 09:51:04AM +0800, fengchengwen wrote: > Hi Bruce, > > I think escape the string at begin (following function) seem more simple: > rte_tel_data_string > rte_tel_data_add_array_string > rte_tel_data_add_dict_string > > int > rte_tel_data_string(struct rte_t

RE: [PATCH v2 00/13] telemetry JSON escaping and other enhancements

2022-07-26 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Monday, 25 July 2022 18.35 > > This patchset contains fixes for the problem of handling characters > returned by telemetry callbacks which require escaping when encoded in > JSON format. It also includes unit tests to validate th

[PATCH v2 00/13] telemetry JSON escaping and other enhancements

2022-07-25 Thread Bruce Richardson
This patchset contains fixes for the problem of handling characters returned by telemetry callbacks which require escaping when encoded in JSON format. It also includes unit tests to validate the correct encoding in such scenarios and a number of smaller enhancements to telemetry and telemetry test