Re: [PATCH v2 1/1] testsuite: add parameter to emit bitbake dependency graph

["'MOESSBAUER, Felix' via isar-users" <isar-users@googlegroups.com>]

On Mon, 2025-12-15 at 18:10 +0100, Felix Moessbauer wrote:
> When running with this avocado parameter, bitbake is called with "-g"
> prior to executing the build. By that, a dependency graph of what
> bitbake will execute is dumped that tremendously helps in identifying
> what we test. This further helps developers to strip down tests to the
> features that should be tested by locating unwanted tasks that are
> pulled in via transitive dependencies.
> 
> The feature is modeled as a test function itself, so it can later be
> extended to write parser-only tests or graph analysis tests.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
> Changes since v1:
> - working version. v1 was the wrong commit. Sorry
> 
> Felix
> 
>  testsuite/README.md    |  3 +++
>  testsuite/cibase.py    | 14 ++++++++++++++
>  testsuite/cibuilder.py |  4 +++-
>  3 files changed, 20 insertions(+), 1 deletion(-)
> 
> diff --git a/testsuite/README.md b/testsuite/README.md
> index 3b2be5af..7c11bfb8 100644
> --- a/testsuite/README.md
> +++ b/testsuite/README.md
> @@ -27,6 +27,9 @@ $ avocado run ../testsuite/citest.py -t dev --max-parallel-tasks=1
>  
>  ```
>  $ avocado run ../testsuite/citest.py -t single --max-parallel-tasks=1 -p machine=qemuamd64 -p distro=bullseye
> +# Generate bitbake dependency graph as well
> +# The output will be in build_dir/{task-depends-<testname>.dot, pn-buildlist-<testname>}
> +$ avocado run ../testsuite/citest.py -t single --max-parallel-tasks=1 -p machine=qemuamd64 -p distro=bullseye -p depgraph=1
>  ```
>  
>  ## Fast build test
> diff --git a/testsuite/cibase.py b/testsuite/cibase.py
> index 0a006a21..190a0b7d 100755
> --- a/testsuite/cibase.py
> +++ b/testsuite/cibase.py
> @@ -17,10 +17,24 @@ class CIBaseTest(CIBuilder):
>      def perform_build_test(self, targets, should_fail=False, **kwargs):
>          self.configure(**kwargs)
>  
> +        if bool(int(self.params.get('depgraph', default=0))):
> +            self.generate_dependency_graph(targets)

Here we potentially configure twice, but this time without passing the
kwargs. Will be fixed in a v3.

Felix

-- 
Siemens AG
Linux Expert Center
Friedrich-Ludwig-Bauer-Str. 3
85748 Garching, Germany

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/98d0eea384bfb177f39dccd9fc6fdedaf2698168.camel%40siemens.com.