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