[v2,0/8] dts: add testpmd params

Message ID 20240509112057.1167947-1-luca.vizzarro@arm.com (mailing list archive)
Headers
Series dts: add testpmd params |

Message

Luca Vizzarro May 9, 2024, 11:20 a.m. UTC
  Hello,

sending in v2:
- refactored the params module
- strengthened typing of the params module
- moved the params module into its own package
- refactored EalParams and TestPmdParams and
  moved under the params package
- reworked interactions between nodes and shells
- refactored imports leading to circular dependencies

Best,
Luca

---
Depends-on: series-31896 ("dts: update mypy and clean up")
---

Luca Vizzarro (8):
  dts: add params manipulation module
  dts: use Params for interactive shells
  dts: refactor EalParams
  dts: remove module-wide imports
  dts: add testpmd shell params
  dts: use testpmd params for scatter test suite
  dts: rework interactive shells
  dts: use Unpack for type checking and hinting

 dts/framework/params/__init__.py              | 274 ++++++++
 dts/framework/params/eal.py                   |  50 ++
 dts/framework/params/testpmd.py               | 608 ++++++++++++++++++
 dts/framework/params/types.py                 | 133 ++++
 dts/framework/remote_session/__init__.py      |   5 +-
 dts/framework/remote_session/dpdk_shell.py    | 104 +++
 .../remote_session/interactive_shell.py       |  83 ++-
 dts/framework/remote_session/python_shell.py  |   4 +-
 dts/framework/remote_session/testpmd_shell.py | 102 ++-
 dts/framework/runner.py                       |   4 +-
 dts/framework/test_suite.py                   |   5 +-
 dts/framework/testbed_model/__init__.py       |   7 -
 dts/framework/testbed_model/node.py           |  36 +-
 dts/framework/testbed_model/os_session.py     |  38 +-
 dts/framework/testbed_model/sut_node.py       | 182 +-----
 .../testbed_model/traffic_generator/scapy.py  |   6 +-
 dts/tests/TestSuite_hello_world.py            |   9 +-
 dts/tests/TestSuite_pmd_buffer_scatter.py     |  21 +-
 dts/tests/TestSuite_smoke_tests.py            |   4 +-
 19 files changed, 1296 insertions(+), 379 deletions(-)
 create mode 100644 dts/framework/params/__init__.py
 create mode 100644 dts/framework/params/eal.py
 create mode 100644 dts/framework/params/testpmd.py
 create mode 100644 dts/framework/params/types.py
 create mode 100644 dts/framework/remote_session/dpdk_shell.py
  

Comments

Nicholas Pratte May 22, 2024, 3:59 p.m. UTC | #1
In addition to the pmd_scatter suite, I refactored my jumboframes
suite to use this new module for testing purposes; everything works
great, and the format looks much better too.

Tested-by: Nicholas Pratte <npratte@iol.unh.edu>
Reviewed-by: Nicholas Pratte <npratte@iol.unh.edu>


On Thu, May 9, 2024 at 7:21 AM Luca Vizzarro <luca.vizzarro@arm.com> wrote:
>
> Hello,
>
> sending in v2:
> - refactored the params module
> - strengthened typing of the params module
> - moved the params module into its own package
> - refactored EalParams and TestPmdParams and
>   moved under the params package
> - reworked interactions between nodes and shells
> - refactored imports leading to circular dependencies
>
> Best,
> Luca
>
> ---
> Depends-on: series-31896 ("dts: update mypy and clean up")
> ---
>
> Luca Vizzarro (8):
>   dts: add params manipulation module
>   dts: use Params for interactive shells
>   dts: refactor EalParams
>   dts: remove module-wide imports
>   dts: add testpmd shell params
>   dts: use testpmd params for scatter test suite
>   dts: rework interactive shells
>   dts: use Unpack for type checking and hinting
>
>  dts/framework/params/__init__.py              | 274 ++++++++
>  dts/framework/params/eal.py                   |  50 ++
>  dts/framework/params/testpmd.py               | 608 ++++++++++++++++++
>  dts/framework/params/types.py                 | 133 ++++
>  dts/framework/remote_session/__init__.py      |   5 +-
>  dts/framework/remote_session/dpdk_shell.py    | 104 +++
>  .../remote_session/interactive_shell.py       |  83 ++-
>  dts/framework/remote_session/python_shell.py  |   4 +-
>  dts/framework/remote_session/testpmd_shell.py | 102 ++-
>  dts/framework/runner.py                       |   4 +-
>  dts/framework/test_suite.py                   |   5 +-
>  dts/framework/testbed_model/__init__.py       |   7 -
>  dts/framework/testbed_model/node.py           |  36 +-
>  dts/framework/testbed_model/os_session.py     |  38 +-
>  dts/framework/testbed_model/sut_node.py       | 182 +-----
>  .../testbed_model/traffic_generator/scapy.py  |   6 +-
>  dts/tests/TestSuite_hello_world.py            |   9 +-
>  dts/tests/TestSuite_pmd_buffer_scatter.py     |  21 +-
>  dts/tests/TestSuite_smoke_tests.py            |   4 +-
>  19 files changed, 1296 insertions(+), 379 deletions(-)
>  create mode 100644 dts/framework/params/__init__.py
>  create mode 100644 dts/framework/params/eal.py
>  create mode 100644 dts/framework/params/testpmd.py
>  create mode 100644 dts/framework/params/types.py
>  create mode 100644 dts/framework/remote_session/dpdk_shell.py
>
> --
> 2.34.1
>