From: Luca Vizzarro <Luca.Vizzarro@arm.com>
To: Dean Marx <dmarx@iol.unh.edu>
Cc: dev@dpdk.org, "Paul Szczepanek" <paul.szczepanek@arm.com>,
"Patrick Robb" <probb@iol.unh.edu>,
"Tomáš Ďurovec" <tomas.durovec@pantheon.tech>
Subject: Re: [PATCH v2 5/7] dts: add support for externally compiled DPDK
Date: Tue, 29 Oct 2024 11:43:00 +0000 [thread overview]
Message-ID: <ceea7c36-d9d8-4b56-a8b4-1ac82f0a8548@arm.com> (raw)
In-Reply-To: <CABD7UXOmGgg8KxpTmJJF3WBgV0X7KzVqojrijK-netFuCnR00Q@mail.gmail.com>
On 25/10/2024 19:29, Dean Marx wrote:
> Just wondering, if the user wants to run DTS using the configuration
> yaml file or the environment variables instead of the command line args,
> are they supposed to run main.py by itself after setup? Because right
> now, it won't run without the --tarball or --dpdk-tree args, even if
> it's specified in the config file. It seems like it just ignores
> whatever was specified in the command line if it's specified in the yaml
> file, but won't execute without the command line args either.
Hi Dean,
If I understood your query correctly, I can't seem to replicate your
issue, and I need some steps so that I fully understand.
My configuration file is set as:
dpdk_build:
dpdk_tree: /home/luca/dpdk
remote: true
precompiled_build_dir: build
and running:
./main.py --config-file my-conf.yaml
works just as expected. It re-uses my pre-build DPDK repository on my
SUT node. We tried every setup:
(tree, tarball)*(local, remote)*(precompiled, build_options)
and we were able to get it to work without issue. In case I
misunderstood your problem I just attempted to call:
(cd ../ && git archive --format=tar.gz -o dts/dpdk-HEAD.tar.gz \
--prefix dpdk-HEAD/ HEAD)
./main.py --config-file my-conf.yaml --tarball dpdk.tar.gz
and as expected it failed because my configuration was assuming a
precompiled build dir in my tarball, but it copied and extracted the
tarball on the remote correctly. Changing from precompiled to build,
made it fully work.
Looking forward to hearing back from you.
Best,
Luca
next prev parent reply other threads:[~2024-10-29 11:43 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-30 16:01 [PATCH 0/7] DTS external DPDK build Tomáš Ďurovec
2024-09-30 16:01 ` [PATCH 1/7] dts: rename build target to " Tomáš Ďurovec
2024-09-30 16:01 ` [PATCH 2/7] dts: one dpdk build per test run Tomáš Ďurovec
2024-09-30 16:02 ` [PATCH 3/7] dts: fix remote session file transfer vars Tomáš Ďurovec
2024-09-30 16:02 ` [PATCH 4/7] dts: add the ability to copy directories via remote Tomáš Ďurovec
2024-09-30 16:02 ` [PATCH 5/7] dts: add support for externally compiled DPDK Tomáš Ďurovec
2024-09-30 16:02 ` [PATCH 6/7] doc: update argument options for external DPDK build Tomáš Ďurovec
2024-09-30 16:02 ` [PATCH 7/7] dts: remove git ref option Tomáš Ďurovec
2024-10-21 13:49 ` [PATCH v2 0/7] DTS external DPDK build Luca Vizzarro
2024-10-21 13:49 ` [PATCH v2 1/7] dts: rename build target to " Luca Vizzarro
2024-10-25 18:05 ` Dean Marx
2024-10-29 1:29 ` Patrick Robb
2024-10-21 13:49 ` [PATCH v2 2/7] dts: enforce one dpdk build per test run Luca Vizzarro
2024-10-25 18:08 ` Dean Marx
2024-10-29 1:29 ` Patrick Robb
2024-10-21 13:49 ` [PATCH v2 3/7] dts: fix remote session file transfer vars Luca Vizzarro
2024-10-21 13:49 ` [PATCH v2 4/7] dts: enable copying directories to and from nodes Luca Vizzarro
2024-10-25 18:12 ` Dean Marx
2024-10-29 1:07 ` Patrick Robb
2024-10-29 12:00 ` Luca Vizzarro
2024-10-21 13:49 ` [PATCH v2 5/7] dts: add support for externally compiled DPDK Luca Vizzarro
2024-10-25 18:29 ` Dean Marx
2024-10-29 11:43 ` Luca Vizzarro [this message]
2024-10-29 15:48 ` Dean Marx
2024-10-29 19:52 ` Dean Marx
2024-10-29 21:12 ` Luca Vizzarro
2024-10-29 1:19 ` Patrick Robb
2024-10-29 11:48 ` Luca Vizzarro
2024-10-29 19:59 ` Patrick Robb
2024-10-21 13:49 ` [PATCH v2 6/7] doc: update argument options for external DPDK build Luca Vizzarro
2024-10-25 18:30 ` Dean Marx
2024-10-29 1:23 ` Patrick Robb
2024-10-29 12:35 ` Luca Vizzarro
2024-10-29 20:04 ` Patrick Robb
2024-10-21 13:49 ` [PATCH v2 7/7] dts: remove git ref option Luca Vizzarro
2024-10-25 18:31 ` Dean Marx
2024-10-29 1:28 ` Patrick Robb
2024-10-29 11:51 ` Luca Vizzarro
2024-10-21 22:39 ` [PATCH v2 0/7] DTS external DPDK build Dean Marx
2024-10-22 7:27 ` Paul Szczepanek
2024-10-22 14:56 ` Luca Vizzarro
2024-10-24 17:14 ` Dean Marx
2024-10-21 22:46 ` [PATCH v3 " Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 1/7] dts: rename build target to " Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 2/7] dts: enforce one dpdk build per test run Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 3/7] dts: change remote and local paths objects Luca Vizzarro
2024-10-25 18:10 ` Dean Marx
2024-10-29 1:29 ` Patrick Robb
2024-10-21 22:46 ` [PATCH v3 4/7] dts: enable copying directories to and from nodes Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 5/7] dts: add support for externally compiled DPDK Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 6/7] doc: update argument options for external DPDK build Luca Vizzarro
2024-10-21 22:46 ` [PATCH v3 7/7] dts: remove git ref option 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=ceea7c36-d9d8-4b56-a8b4-1ac82f0a8548@arm.com \
--to=luca.vizzarro@arm.com \
--cc=dev@dpdk.org \
--cc=dmarx@iol.unh.edu \
--cc=paul.szczepanek@arm.com \
--cc=probb@iol.unh.edu \
--cc=tomas.durovec@pantheon.tech \
/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).