DPDK patches and discussions
 help / color / mirror / Atom feed
From: Luca Vizzarro <Luca.Vizzarro@arm.com>
To: dev@dpdk.org
Cc: Luca Vizzarro <luca.vizzarro@arm.com>,
	Patrick Robb <probb@iol.unh.edu>,
	Paul Szczepanek <paul.szczepanek@arm.com>
Subject: [PATCH v4 0/7] dts: add per-test-suite configuration
Date: Mon,  3 Mar 2025 16:57:02 +0200	[thread overview]
Message-ID: <20250303145709.126126-1-Luca.Vizzarro@arm.com> (raw)
In-Reply-To: <20241108134532.130681-1-luca.vizzarro@arm.com>

From: Luca Vizzarro <luca.vizzarro@arm.com>

Hi there,

sending in v4 for the per-test-suite configuration patchset.

v4:
- rebased
- Changed implementation:
  - test suites configurations are now provided
    through a dedicated optional configuration file.
  - running multiple test runs hinders flexibility in loading
    configuration, therefore DTS now accepts only one test run.
  - the tests folder is no longer a package as this is no longer
    imported directly. The implementation now relies on the test
    suite globbing imports, then the configuration class is referenced
    through the annotation for TestSuite.config.
  - as a consequence of the above, a code generator is no longer needed.
- improved the way configuration errors are displayed.
- adding the test suites to the docs resulted in several warnings,
  minimally required fixes are added.

Best,
Luca

Luca Vizzarro (7):
  dts: add tests package to API docs
  dts: amend test suites docstring
  dts: fix smoke tests docstring
  dts: update autodoc sorting order
  dts: run only one test run per execution
  dts: add per-test-suite configuration
  dts: improve configuration errors

 doc/api/dts/index.rst                         |   1 +
 doc/api/dts/tests.TestSuite_blocklist.rst     |   8 ++
 .../dts/tests.TestSuite_checksum_offload.rst  |   8 ++
 doc/api/dts/tests.TestSuite_dual_vlan.rst     |   8 ++
 .../dts/tests.TestSuite_dynamic_config.rst    |   8 ++
 .../tests.TestSuite_dynamic_queue_conf.rst    |   8 ++
 doc/api/dts/tests.TestSuite_hello_world.rst   |   8 ++
 doc/api/dts/tests.TestSuite_l2fwd.rst         |   8 ++
 doc/api/dts/tests.TestSuite_mac_filter.rst    |   8 ++
 doc/api/dts/tests.TestSuite_mtu.rst           |   8 ++
 .../tests.TestSuite_pmd_buffer_scatter.rst    |   8 ++
 ...tSuite_port_restart_config_persistency.rst |   8 ++
 .../dts/tests.TestSuite_promisc_support.rst   |   8 ++
 .../dts/tests.TestSuite_queue_start_stop.rst  |   8 ++
 doc/api/dts/tests.TestSuite_smoke_tests.rst   |   8 ++
 doc/api/dts/tests.TestSuite_softnic.rst       |   8 ++
 doc/api/dts/tests.TestSuite_uni_pkt.rst       |   8 ++
 doc/api/dts/tests.TestSuite_vlan.rst          |   8 ++
 doc/api/dts/tests.rst                         |  11 ++
 doc/guides/conf.py                            |   1 +
 doc/guides/tools/dts.rst                      |  25 ++--
 dts/.gitignore                                |   2 +-
 dts/framework/config/__init__.py              | 112 ++++++++++-------
 dts/framework/config/test_run.py              |  53 ++++++--
 dts/framework/runner.py                       |  33 +++--
 dts/framework/settings.py                     |  33 +++--
 dts/framework/test_run.py                     |  23 ++--
 dts/framework/test_suite.py                   |  25 ++--
 dts/test_run.example.yaml                     |  43 +++++++
 dts/test_runs.example.yaml                    |  43 -------
 dts/tests/TestSuite_dynamic_config.py         |  10 +-
 dts/tests/TestSuite_hello_world.py            |  14 ++-
 dts/tests/TestSuite_mac_filter.py             |  42 +++----
 dts/tests/TestSuite_mtu.py                    | 114 +++++++++---------
 dts/tests/TestSuite_smoke_tests.py            |   2 -
 dts/tests_config.example.yaml                 |   2 +
 36 files changed, 495 insertions(+), 230 deletions(-)
 create mode 100644 doc/api/dts/tests.TestSuite_blocklist.rst
 create mode 100644 doc/api/dts/tests.TestSuite_checksum_offload.rst
 create mode 100644 doc/api/dts/tests.TestSuite_dual_vlan.rst
 create mode 100644 doc/api/dts/tests.TestSuite_dynamic_config.rst
 create mode 100644 doc/api/dts/tests.TestSuite_dynamic_queue_conf.rst
 create mode 100644 doc/api/dts/tests.TestSuite_hello_world.rst
 create mode 100644 doc/api/dts/tests.TestSuite_l2fwd.rst
 create mode 100644 doc/api/dts/tests.TestSuite_mac_filter.rst
 create mode 100644 doc/api/dts/tests.TestSuite_mtu.rst
 create mode 100644 doc/api/dts/tests.TestSuite_pmd_buffer_scatter.rst
 create mode 100644 doc/api/dts/tests.TestSuite_port_restart_config_persistency.rst
 create mode 100644 doc/api/dts/tests.TestSuite_promisc_support.rst
 create mode 100644 doc/api/dts/tests.TestSuite_queue_start_stop.rst
 create mode 100644 doc/api/dts/tests.TestSuite_smoke_tests.rst
 create mode 100644 doc/api/dts/tests.TestSuite_softnic.rst
 create mode 100644 doc/api/dts/tests.TestSuite_uni_pkt.rst
 create mode 100644 doc/api/dts/tests.TestSuite_vlan.rst
 create mode 100644 doc/api/dts/tests.rst
 create mode 100644 dts/test_run.example.yaml
 delete mode 100644 dts/test_runs.example.yaml
 create mode 100644 dts/tests_config.example.yaml

-- 
2.43.0


  parent reply	other threads:[~2025-03-03 14:57 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-06 16:13 [PATCH] " Luca Vizzarro
2024-09-27 17:45 ` Jeremy Spewock
2024-11-08 13:38 ` [PATCH v2 0/4] " Luca Vizzarro
2024-11-08 13:38   ` [PATCH v2 1/4] dts: add tests package to API docs Luca Vizzarro
2024-11-08 13:38   ` [PATCH v2 2/4] dts: fix smoke tests docstring Luca Vizzarro
2024-11-12 20:03     ` Dean Marx
2024-11-08 13:38   ` [PATCH v2 3/4] dts: add per-test-suite configuration Luca Vizzarro
2024-11-08 13:38   ` [PATCH v2 4/4] dts: update autodoc sorting order Luca Vizzarro
2024-11-08 13:45 ` [PATCH v3 0/4] dts: add per-test-suite configuration Luca Vizzarro
2024-11-08 13:45   ` [PATCH v3 1/4] dts: add tests package to API docs Luca Vizzarro
2024-11-12 19:55     ` Dean Marx
2024-11-08 13:45   ` [PATCH v3 2/4] dts: fix smoke tests docstring Luca Vizzarro
2024-11-08 13:45   ` [PATCH v3 3/4] dts: add per-test-suite configuration Luca Vizzarro
2024-11-13 21:44     ` Dean Marx
2024-11-14 12:39       ` Luca Vizzarro
2024-11-19 16:31         ` Dean Marx
2024-11-19 16:41           ` Luca Vizzarro
2024-11-08 13:45   ` [PATCH v3 4/4] dts: update autodoc sorting order Luca Vizzarro
2024-11-12 20:04     ` Dean Marx
2025-03-03 14:57   ` Luca Vizzarro [this message]
2025-03-03 14:57     ` [PATCH v4 1/7] dts: add tests package to API docs Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 2/7] dts: amend test suites docstring Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 3/7] dts: fix smoke tests docstring Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 4/7] dts: update autodoc sorting order Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 5/7] dts: run only one test run per execution Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 6/7] dts: add per-test-suite configuration Luca Vizzarro
2025-03-03 14:57     ` [PATCH v4 7/7] dts: improve configuration errors Luca Vizzarro

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=20250303145709.126126-1-Luca.Vizzarro@arm.com \
    --to=luca.vizzarro@arm.com \
    --cc=dev@dpdk.org \
    --cc=paul.szczepanek@arm.com \
    --cc=probb@iol.unh.edu \
    /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).