From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B392D4333A; Wed, 15 Nov 2023 16:56:16 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A3C9840A7A; Wed, 15 Nov 2023 16:56:16 +0100 (CET) Received: from mail-oi1-f177.google.com (mail-oi1-f177.google.com [209.85.167.177]) by mails.dpdk.org (Postfix) with ESMTP id 6B993402B7 for ; Wed, 15 Nov 2023 16:56:14 +0100 (CET) Received: by mail-oi1-f177.google.com with SMTP id 5614622812f47-3b3f55e1bbbso4499491b6e.2 for ; Wed, 15 Nov 2023 07:56:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1700063773; x=1700668573; darn=dpdk.org; h=cc:to:subject:message-id:date:from:mime-version:from:to:cc:subject :date:message-id:reply-to; bh=bDoQBnh8FLfWqUmEIBafNwRBuDqLsSyRYARNgctTD+Y=; b=GWoLXdMajffBJ0VGMtlCJ1schmvDpQQCKIweapsxc/s/zr1kEvUsIXx+ekL/HJwitz LgAUNCEgoxMscyEJJnHHjmvNstbKJJvck1LYbg67rgBzUoCcf1iXyhcfwNpQZjearu2H /FppVfnQe2nZ3NIQmNiYlVQb2EfAj+T2e4W1A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700063773; x=1700668573; h=cc:to:subject:message-id:date:from:mime-version:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=bDoQBnh8FLfWqUmEIBafNwRBuDqLsSyRYARNgctTD+Y=; b=eZ1OrneFc6XRQksKbv8UJvdWHuMGWA3EeuLFTh5b735lQfauSeU8RZg+yNGrg8+eiN QpB05rZcHMKD4KdJo/aSGsM/6Iq+z/ajX3U4kwZTEtW6bom0u8cQCNnU5iQZum1aUGSJ KVu22pnF0Th+5oJUQMpJpK9zAWiGvFyJlISVUkayJZar0DbRZhRz1ylozyopZxg4cKzJ iinlLUCyV4Br9h5TL4BXbqAfP0ASXldue+j7PijL8YnmWAOWh5Dv6ZxsY2j5KZd2fFf2 ZRthrz3B8jI9H85i98Y/KqCYUfK/kN2FNgubT226Jb7JvR9PJZ26MpDcpBaL8F1wEHU8 2X9A== X-Gm-Message-State: AOJu0YyLFeDzqreshoebuq7GeGVj5ubVkOAzR6Ks1pt2N3ctcYFFEOLm zFBADahE4488COLdV8YmgTwKVzJJ0iumAnHKoCEvKQkDZj2CnY9587s= X-Google-Smtp-Source: AGHT+IHp3SO4o3auoYqj2AMcbmXpoVGjwkMA/BxoeOVrDPi89OmFeN8L0WHs1S/lX51AZHuV+PsdDc0aTYePrFLG3/Y= X-Received: by 2002:a05:6808:1827:b0:3a7:6d4a:fd78 with SMTP id bh39-20020a056808182700b003a76d4afd78mr16220434oib.24.1700063773468; Wed, 15 Nov 2023 07:56:13 -0800 (PST) MIME-Version: 1.0 From: Patrick Robb Date: Wed, 15 Nov 2023 10:56:02 -0500 Message-ID: Subject: DTS Working Group Meeting Minutes - November 15, 2023 To: dts@dpdk.org Cc: ci@dpdk.org, dev@dpdk.org Content-Type: multipart/alternative; boundary="000000000000504814060a32f11b" X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org --000000000000504814060a32f11b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable November 15, 2023 ----------------------------- Attendees * Patrick Robb * Jeremy Spewock * Juraj Linke=C5=A1 * Paul Szczepanek * Honnappa Nagarahalli * Luca Vizzarro ----------------------------- Agenda * Additions to the agenda * Patch discussions * DTS Developer documentation * 24.03 roadmap ----------------------------- Minutes ----------------------------- Patch discussions * 1) DTS API document auto generation 1. Juraj submitted a V7 this morning 2. There are reviews on the ARM side, and Jeremy is also providing a review 3. Patch has been split into the docstring section, and the meson sectio= n 1. ARM folks will provide reviews for all the docstring change and code changes this week 2. Jeremy can dedicate the remainder of his week (if needed) on this review 3. Has Bruce provided feedback on Meson updates? This patchseries only got released today, but we can ping him if we need. * 2) Scatter test suite along with TG packet manipulation and verification 1. Jeremy submitted a patch making DTS handle device binding on the SUT 2. Patrick did a review for the testsuite last night, will follow up with reviews for the other patches in the series today 3. Old DTS has an assumption that the packet at index 0 received at the rx port is the packet you want to validate against, and this appears to reliably be true. 1. Lincoln suggested storing some information about the packet and then filtering out rx packets at the end based on this information, or whitelisting packets based on some attribute unique to the packet being sen= t 2. Scapy might have some support for sending packets, tracking which was sent, and validating * 3) Pending patches to merge 1. Dockerfile for DTS - https://patches.dpdk.org/project/dpdk/patch/20221103134633.446646-1-juraj.l= inkes@pantheon.tech/ 1. Jeremy is going to update this and resubmit this week 2. The poetry version from apt is too old given how we=E2=80=99ve wri= tten our poetry file, so you need to switch to installing from PIP in the dockerfile ----------------------------- DTS Developer documentation * Luca is looking into updates that need to be made for the documentation * He notes that the setup dts.rst doc needs to be updated to reflect change in poetry, as there has been an update and a ./dts package folder is expected where you run poetry. We can either add this, or add the =E2=80=93= no-root flag to the poetry install command as written in the documentation. * Setting up the configuration is rather confusing. It might be possible to use a tool to translate the JSON_Schema to a more human readable format. * If you run ./main.py without =E2=80=93help, automatically print the he= lp information if you have an exception * Refer to DPDK system requirements in documentation, and explain that the SUT needs to be setup accordingly. * The way that logging and error reporting is not very user friendly at the moment, as it does not explain what=E2=80=99s wrong ----------------------------- 24.03 Roadmap * Will go over at the next meeting ----------------------------- Action Items * Juraj will review the DTS testcases list doc Patrick made a few weeks ago, will discuss at the next meeting ----------------------------- Any other business * Next Meeting: will determine at the CI meeting on November 30 --000000000000504814060a32f11b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
November 15, 2023

---------------------= --------
Attendees
* Patrick Robb
* Jeremy Spewock
* Juraj Link= e=C5=A1
* Paul Szczepanek
* Honnappa Nagarahalli
* Luca Vizzarro

-----------------------------
Agenda
* Additi= ons to the agenda
* Patch discussions
* DTS Developer documentation* 24.03 roadmap

-----------------------------Minutes

-----------------------------
Patch di= scussions
* 1) DTS API document auto generation
=C2=A0 =C2=A01. Juraj= submitted a V7 this morning
=C2=A0 =C2=A02. There are reviews on the AR= M side, and Jeremy is also providing a review
=C2=A0 =C2=A03. Patch has = been split into the docstring section, and the meson section
=C2=A0 =C2= =A0 =C2=A0 1. ARM folks will provide reviews for all the docstring change a= nd code changes this week
=C2=A0 =C2=A0 =C2=A0 2. Jeremy can dedicate th= e remainder of his week (if needed) on this review
=C2=A0 =C2=A0 =C2=A0 = 3. Has Bruce provided feedback on Meson updates? This patchseries only got = released today, but we can ping him if we need.
* 2) Scatter test suite= along with TG packet manipulation and verification
=C2=A0 =C2=A01. Jere= my submitted a patch making DTS handle device binding on the SUT
=C2=A0 = =C2=A02. Patrick did a review for the testsuite last night, will follow up = with reviews for the other patches in the series today
=C2=A0 =C2=A03. O= ld DTS has an assumption that the packet at index 0 received at the rx port= is the packet you want to validate against, and this appears to reliably b= e true.
=C2=A0 =C2=A0 =C2=A0 1. Lincoln suggested storing some in= formation about the packet and then filtering out rx packets at the end bas= ed on this information, or whitelisting packets based on some attribute uni= que to the packet being sent
=C2=A0 =C2=A0 =C2=A0 2. Scapy might have so= me support for sending packets, tracking which was sent, and validating
= * 3) Pending patches to merge
=C2=A0 =C2=A01. Dockerfile for DTS - https://patches.dpdk.org/project/dpdk/patch/202= 21103134633.446646-1-juraj.linkes@pantheon.tech/
=C2=A0 =C2=A0 =C2= =A0 1. Jeremy is going to update this and resubmit this week
=C2=A0 =C2= =A0 =C2=A0 2. The poetry version from apt is too old given how we=E2=80=99v= e written our poetry file, so you need to switch to installing from PIP in = the dockerfile

-----------------------------
DT= S Developer documentation
* Luca is looking into updates that need to be= made for the documentation
=C2=A0 =C2=A0* He notes that the setup dts.r= st doc needs to be updated to reflect change in poetry, as there has been a= n update and a ./dts package folder is expected where you run poetry. We ca= n either add this, or add the =E2=80=93no-root flag to the poetry install c= ommand as written in the documentation.
=C2=A0 =C2=A0* Setting up the c= onfiguration is rather confusing. It might be possible to use a tool to tra= nslate the JSON_Schema to a more human readable format.
=C2=A0 =C2=A0* = If you run ./main.py without =E2=80=93help, automatically print the help in= formation if you have an exception
=C2=A0 =C2=A0* Refer to DPDK system r= equirements in documentation, and explain that the SUT needs to be setup ac= cordingly.
=C2=A0 =C2=A0* The way that logging and error reporting is n= ot very user friendly at the moment, as it does not explain what=E2=80=99s = wrong

-----------------------------
24.03 Roadm= ap
* =C2=A0Will go over at the next meeting


-----------------------------
Action Items
* Juraj wi= ll review the DTS testcases list doc Patrick made a few weeks ago, will dis= cuss at the next meeting

-------------------------= ----
Any other business
* Next Meeting: will determine at the CI meet= ing on November 30
--000000000000504814060a32f11b--