On Mon, Oct 21, 2024 at 9:50 AM Luca Vizzarro <luca.vizza...@arm.com> wrote:
> From: Tomáš Ďurovec <tomas.duro...@pantheon.tech> > > Enable the user to use either a DPDK source tree directory or a > tarball, with and without a pre-built build directory. These can be > stored on either SUT node or the DTS host. The DPDK build setup or the > pre-built binaries can be specified through the configuration file, > the command line arguments or environment variables. > > Signed-off-by: Tomáš Ďurovec <tomas.duro...@pantheon.tech> > Signed-off-by: Luca Vizzarro <luca.vizza...@arm.com> > --- > dts/conf.yaml | 24 +- > dts/framework/config/__init__.py | 123 ++++++- > dts/framework/config/conf_yaml_schema.json | 62 +++- > dts/framework/config/types.py | 17 +- > dts/framework/exception.py | 4 +- > dts/framework/remote_session/dpdk_shell.py | 2 +- > dts/framework/runner.py | 8 +- > dts/framework/settings.py | 200 +++++++++-- > dts/framework/test_result.py | 23 +- > dts/framework/testbed_model/node.py | 22 +- > dts/framework/testbed_model/os_session.py | 63 +++- > dts/framework/testbed_model/posix_session.py | 39 ++- > dts/framework/testbed_model/sut_node.py | 351 +++++++++++++------ > 13 files changed, 732 insertions(+), 206 deletions(-) > > diff --git a/dts/conf.yaml b/dts/conf.yaml > index 814744a1fc..8a65a481d6 100644 > --- a/dts/conf.yaml > +++ b/dts/conf.yaml > @@ -5,12 +5,24 @@ > test_runs: > # define one test run environment > - dpdk_build: > - arch: x86_64 > - os: linux > - cpu: native > - # the combination of the following two makes CC="ccache gcc" > - compiler: gcc > - compiler_wrapper: ccache > + # dpdk_tree: Commented out because `tarball` is defined. > Should this key name be extended to clarify the desired value is a path, and whether it is absolute or relative? +1 to Dean's concerns about using the tarball without --tarball flag if dpdk_tree is commented out and tarball is defined. Reviewed-by: Patrick Robb <pr...@iol.unh.edu>
