Hey Robin, my name is Mathew Labrecque and I work over at the UNH InterOperability Laboratory on the Open-Source Team.
I wanted to email you regarding the ARM cross-compile tests that failed on the patch. While it says the tests were run on Ubuntu 20.04, these tests were actually performed on Ubuntu 24.04. This was due to an internal error where we hadn't properly updated the labels for the tests. Cheers, Matthew On Thu, Jan 8, 2026 at 5:50 AM Robin Jarry <[email protected]> wrote: > Currently, adding TEL_DICT containers in TEL_ARRAY_CONTAINER containers > is not supported by rte_tel_data_add_array_container. It wrongfully > returns -EINVAL where it should allow it. > > This is supported by the JSON formatter. > > Allow the same value types than rte_tel_data_add_dict_container(). > > Add test case to verify it works as expected. > > Fixes: c933bb5177ca ("telemetry: support array values in data object") > Cc: [email protected] > > Signed-off-by: Robin Jarry <[email protected]> > --- > > Notes: > v2: added test case > > app/test/test_telemetry_data.c | 21 +++++++++++++++++++++ > lib/telemetry/telemetry_data.c | 6 +++--- > 2 files changed, 24 insertions(+), 3 deletions(-) > > diff --git a/app/test/test_telemetry_data.c > b/app/test/test_telemetry_data.c > index 59898ff5e983..e904b5a7f210 100644 > --- a/app/test/test_telemetry_data.c > +++ b/app/test/test_telemetry_data.c > @@ -405,6 +405,26 @@ test_array_with_array_uint_hex_values_nopadding(void) > return CHECK_OUTPUT("[[\"0x888\"],[\"0x8888\"]]"); > } > > +static int > +test_array_with_dict_values(void) > +{ > + rte_tel_data_start_array(&response_data, RTE_TEL_CONTAINER); > + > + struct rte_tel_data *d1 = rte_tel_data_alloc(); > + rte_tel_data_start_dict(d1); > + rte_tel_data_add_dict_string(d1, "name", "foo"); > + rte_tel_data_add_dict_uint(d1, "size", 42); > + rte_tel_data_add_array_container(&response_data, d1, 0); > + > + struct rte_tel_data *d2 = rte_tel_data_alloc(); > + rte_tel_data_start_dict(d2); > + rte_tel_data_add_dict_string(d2, "name", "bar"); > + rte_tel_data_add_dict_uint(d2, "size", 666); > + rte_tel_data_add_array_container(&response_data, d2, 0); > + > + return > CHECK_OUTPUT("[{\"name\":\"foo\",\"size\":42},{\"name\":\"bar\",\"size\":666}]"); > +} > + > static int > test_case_array_u64(void) > { > @@ -597,6 +617,7 @@ telemetry_data_autotest(void) > test_array_with_array_string_values, > test_array_with_array_uint_hex_values_padding, > test_array_with_array_uint_hex_values_nopadding, > + test_array_with_dict_values, > test_string_char_escaping, > test_array_char_escaping, > test_dict_char_escaping, > diff --git a/lib/telemetry/telemetry_data.c > b/lib/telemetry/telemetry_data.c > index 0354a0654804..0a006559ab42 100644 > --- a/lib/telemetry/telemetry_data.c > +++ b/lib/telemetry/telemetry_data.c > @@ -96,10 +96,10 @@ int > rte_tel_data_add_array_container(struct rte_tel_data *d, > struct rte_tel_data *val, int keep) > { > - if (d->type != TEL_ARRAY_CONTAINER || > - (val->type != TEL_ARRAY_UINT > + if (d->type != TEL_ARRAY_CONTAINER || (val->type != TEL_ARRAY_UINT > && val->type != TEL_ARRAY_INT > - && val->type != TEL_ARRAY_STRING)) > + && val->type != TEL_ARRAY_STRING > + && val->type != TEL_DICT)) > return -EINVAL; > if (d->data_len >= RTE_TEL_MAX_ARRAY_ENTRIES) > return -ENOSPC; > -- > 2.52.0 > > -- Matthew Labrecque Technical Intern, Open Source Team UNH Interoperability Labs 21 Madbury Rd, Suite 100, Durham, NH 03824 www.iol.unh.edu
