On Wed, 2026-01-14 at 18:54 +0000, Vyacheslav Yurkov via lists.openembedded.org wrote: > From: Vyacheslav Yurkov <[email protected]> > > The test orignally was written under assumption that poky distro is > used. When poky-altcft is used for example, then systemd is already set > in DISTRO_FETURES, which the test did not expect.
Hi, The commit title here still talks about determinism. The test case was deterministic though - previously it would always fail if systemd was already in DISTRO_FEATURES. I think I misunderstood a little last time, but this still needs renaming. I also wonder why we're testing this here. Perhaps this can be turned into a more generic test that REQUIRED_DISTRO_FEATURES behaves as expected - we can use the overlayfs bbclass as part of that as we know it sets REQUIRED_DISTRO_FEATURES. > > Signed-off-by: Vyacheslav Yurkov <[email protected]> > --- > meta/lib/oeqa/selftest/cases/overlayfs.py | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/meta/lib/oeqa/selftest/cases/overlayfs.py > b/meta/lib/oeqa/selftest/cases/overlayfs.py > index 580fbdcb9c..3e55e97927 100644 > --- a/meta/lib/oeqa/selftest/cases/overlayfs.py > +++ b/meta/lib/oeqa/selftest/cases/overlayfs.py > @@ -5,7 +5,7 @@ > # > > from oeqa.selftest.case import OESelftestTestCase > -from oeqa.utils.commands import bitbake, runqemu, get_bb_vars > +from oeqa.utils.commands import bitbake, runqemu, get_bb_vars, get_bb_var > from oeqa.core.decorator import OETestTag > from oeqa.core.decorator.data import skipIfNotMachine > > @@ -46,7 +46,8 @@ inherit overlayfs > res = bitbake('core-image-minimal', ignore_status=True) > line = getline(res, "overlayfs-user was skipped: missing required > distro features") > self.assertTrue("overlayfs" in res.output, msg=res.output) > - self.assertTrue("systemd" in res.output, msg=res.output) > + if not "systemd" in get_bb_var('DISTRO_FEATURES'): > + self.assertTrue("systemd" in res.output, msg=res.output) > self.assertTrue("ERROR: Required build target 'core-image-minimal' > has no buildable providers." in res.output, msg=res.output) > > def test_not_all_units_installed(self): Best regards, -- Paul Barker
signature.asc
Description: This is a digitally signed message part
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#229412): https://lists.openembedded.org/g/openembedded-core/message/229412 Mute This Topic: https://lists.openembedded.org/mt/117266572/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
