test suite reviews and discussions
 help / color / mirror / Atom feed
From: Patrick Robb <probb@iol.unh.edu>
To: dev <dev@dpdk.org>
Cc: ci@dpdk.org, dts@dpdk.org
Subject: DTS WG Meeting Minutes - March 27, 2024
Date: Wed, 27 Mar 2024 12:14:56 -0400	[thread overview]
Message-ID: <CAJvnSUBkLL=67oRGk=BWF_D-6mzDZUWL1ovXjBk-webOXwC=sQ@mail.gmail.com> (raw)

#####################################################################
Attendees
* Patrick Robb
* Jeremy Spewock
* Honnappa Nagarahalli
* Luca Vizzarro
* Nicholas Pratte

#####################################################################
Minutes

=====================================================================
General Announcements
* DPDK 24.03 will release this week (releasing today, last I knew)
   * Lab will be sending a few patches which we have reviewed
internally, for the 24.07 release
* UNH Lab will have people at DPDK Summit (likely to be Montreal). So
we can give lab updates and/or DTS updates, or anything else the group
thinks is relevant
   * Honnappa notes it would be good to update the community on new
DTS developments since the last summit
* Ferruh asked on the mailing list about where DTS is for testsuites
which validate the flow api. Patrick responded stating that the 24.07
roadmap contains ethdev testsuites which we will work on soon. But, if
work moves fast and we can get time during the currently development
window, we can write some flow api testcases
   * Honnappa notes that the end goal here is to demonstrate the
effectiveness of the framework and hand off the writing of testsuites
to maintainers/the community at large. So most of our efforts should
be towards that goal (don’t get distracted).
* Will email out the 24.07 roadmap once 24.03 releases

=====================================================================
Patch discussions
* Testpmd statefullness / params class
   * EAL and testpmd params refactored to use this class
   * Original idea was to set some defaults for testpmd to reduce some
load from developers. This does include these defaults.
      * Mix between the existing defaults of testpmd and others in use
in DTS so far
      * Luca is noting that default values on testpmd docs are
incorrect in some cases, some new params are not documented, etc.
      * Do we expose ourselves to some risk by testpmd app changing,
and our testpmd params dataclass becomin “out of sync”
   * May reduce developer user error by setting correct param names,
gives autocomplete to developers for params
   * https://patchwork.dpdk.org/project/dpdk/list/?series=31622
* Scatter:
   * Jeremy working on the secondary testcase which supports MLNX now
   * Also working on the new testpmd context manager which will enable
having multiple testpmd testcases back to back in a testsuite (allows
for waiting for testpmd to fully close)
      1. With most recent run on this patch, the new scatter suites is
running on mellanox NICs.
* Remove xmlrpc server implementation
   * https://bugs.dpdk.org/show_bug.cgi?id=1374
   * When receiving the packets to scapy on ingress port on tester we
need to encode the packets to some string so it can be consumed by the
dts runner. There is a method which translates bytes into base64
encoding, so now that we know about this method we should be all set.
   * Can’t use a “scapy” interactive shell as we originally planned
due to issue with supporting cursor location, just using a python
shell which imports scapy *. Functionally this should be identical to
starting up a literal scapy shell.
* Improved Interactive shell output gathering
   * https://patchwork.dpdk.org/project/dpdk/list/?series=31483
* Nick
   * has tested his patch refactoring how users set hugepage from dts
or from default boot args
   * He is also refactoring the config schema according to Juraj’s
comments, removing information which can be gathered from the system
      1. One caveat is that Juraj suggested we don’t need to set os
value for nodes, but this is required to create the node’s os specific
remote session (which is needed to collect system info)

=====================================================================
Any other business
* UNH people are not changing ticket status when they start working on
a bugzilla ticket, which is confusing people. They will start with
this.
* Possible ticket: Check dependencies on remote nodes.
   * Check python version on TG (required for scapy)
   * Check requirements for DPDK, TG apps, etc. to ensure DTS can run
   * Luca Vizzarrocan create a bugzilla ticket for this

                 reply	other threads:[~2024-03-27 16:15 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAJvnSUBkLL=67oRGk=BWF_D-6mzDZUWL1ovXjBk-webOXwC=sQ@mail.gmail.com' \
    --to=probb@iol.unh.edu \
    --cc=ci@dpdk.org \
    --cc=dev@dpdk.org \
    --cc=dts@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).