Re: [yocto-patches] [yocto-autobuilder-helper][PATCH v2 05/10] scripts: add vcontainer-tarball setup, integration, and publishing
Richard Purdie <[email protected]>
| Newsgroups | org.yoctoproject.lists.yocto-patches |
|---|---|
| Message-ID | <83eab955b09ab5c2bc61fd2a7c0c229f1aea6fd9.camel@linuxfoundation.org> |
On Mon, 2026-06-01 at 16:18 -0700, Tim Orling via lists.yoctoproject.org wrote: > From: Tim Orling <[email protected]> > > Introduce the vcontainer-tarball SDK plumbing used by container build > jobs. The vcontainer-tarball is a meta-virtualization-derived SDK > (modelled after buildtools-tarball) that ships the container build > toolchain so worker jobs do not need to rebuild it for every step. > > * scripts/utils.py: add setup_vcontainer_tarball(), and add an > env_glob keyword argument to setup_tools_tarball() and > enable_tools_tarball() so the vcontainer-tarball can source its > specific environment-setup-ci file rather than the universal > glob. > * scripts/run-config: source the vcontainer-tarball environment > for build-targets / cmds / test-targets / plain-cmds steps, > gated by a new NOVCONTAINER step variable so individual steps > (such as the dashboard indexing step) can opt out independently > of NOBUILDTOOLS. > * scripts/shared-repo-unpack: invoke setup_vcontainer_tarball so > workers extract the SDK during unpack. > * scripts/publish-artefacts: publish the vcontainer-tarball > artefact so downstream test jobs can fetch a stable SDK. > > AI-Generated: Claude Cowork Opus 4.7 > Signed-off-by: Tim Orling <[email protected]> > --- > config.json | 1 + > scripts/publish-artefacts | 5 +++++ > scripts/run-config | 19 +++++++++++++++++++ > scripts/shared-repo-unpack | 1 + > scripts/utils.py | 22 ++++++++++++++++++---- > 5 files changed, 44 insertions(+), 4 deletions(-) > > diff --git a/config.json b/config.json > index 7df4271..16d8a04 100644 > --- a/config.json > +++ b/config.json > @@ -1433,6 +1433,7 @@ > "step3" : { > "shortname" : "Populate/update dashboard site", > "NOBUILDTOOLS" : true, > + "NOVCONTAINER" : true, > "EXTRACMDS" : ["${SCRIPTSDIR}/run-dashboard-index ${HELPERBUILDDIR}/../"] > } > }, Why is this step needing to opt out of a vcontainer? vcontainers are only enabled for specific steps and off by default? I looked further and realised you're basically using a vcontainer everywhere as far as I can tell? Why do we want to have the overhead of doing that outside of the vcontainer builds? For buildtools, it makes sense we need it everywhere except where we configure otherwise but I think for vcontainer, you want to opposite, you only use it in jobs where we actually need/use it? Cheers, Richard