Hello, I've updated my commit messages and sent in the new patches. However, this patch is meant to fix pytest runs, which the autobuilder isn't using yet. You'll also need the update on the autobuilder end from [ https://patchwork.yoctoproject.org/project/yocto/patch/20231201144928.4191-1-alexander.lussier-cul...@savoirfairelinux.com/ | this ] patch to have it use pytest instead of the django tools.
Thanks, Alex From: "Richard Purdie" <[email protected]> To: "Alexander Lussier-Cullen" <[email protected]>, [email protected] Sent: Thursday, November 30, 2023 5:57:39 PM Subject: Re: [Toaster] bitbake: toaster: fix pytest build test execution and test discovery Hi Alexander, On Thu, 2023-11-30 at 08:59 -0500, Alexander Lussier-Cullen wrote: > Signed-off-by: Alexander Lussier-Cullen > <[email protected]> > --- > bitbake/lib/toaster/pytest.ini | 5 +---- > bitbake/lib/toaster/tests/builds/test_core_image_min.py | 2 ++ > 2 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/bitbake/lib/toaster/pytest.ini b/bitbake/lib/toaster/pytest.ini > index f07076b7ea..071c65fcd5 100644 > --- a/bitbake/lib/toaster/pytest.ini > +++ b/bitbake/lib/toaster/pytest.ini > @@ -1,9 +1,5 @@ > # -- FILE: pytest.ini (or tox.ini) > [pytest] > -DJANGO_SETTINGS_MODULE = toastermain.settings_test > - > -python_files = db/test_*.py commands/test_*.py views/test_*.py > browser/test_*.py functional/test_*.py > - > # --create-db - force re creation of the test database > # > https://pytest-django.readthedocs.io/en/latest/database.html#create-db-force-re-creation-of-the-test-database > > > @@ -17,3 +13,4 @@ addopts = --create-db --html="Toaster Tests Report.html" > --self-contained-html > # https://pypi.org/project/pytest-env/ > env = > TOASTER_BUILDSERVER=1 > + DJANGO_SETTINGS_MODULE=toastermain.settings_test I tested this patch and it definitely seems to help. https://autobuilder.yoctoproject.org/typhoon/#/builders/161/builds/12/steps/11/logs/stdio I replied to Alassane earlier about some of the failures in test run but I think this did fix several of them. Can I ask that you tweak the commit message to explain what is being fixed here, just so we have the context in the commit logs please. > diff --git a/bitbake/lib/toaster/tests/builds/test_core_image_min.py > b/bitbake/lib/toaster/tests/builds/test_core_image_min.py > index 9cdaa15f98..ca388eabe7 100644 > --- a/bitbake/lib/toaster/tests/builds/test_core_image_min.py > +++ b/bitbake/lib/toaster/tests/builds/test_core_image_min.py > @@ -10,6 +10,7 @@ > # Ionut Chisanovici, Paul Eggleton and Cristian Iorga > > import os > +import pytest > > from django.db.models import Q > > @@ -21,6 +22,7 @@ from orm.models import CustomImagePackage > from tests.builds.buildtest import BuildTest > > > [email protected]_db(True) > class BuildCoreImageMinimal(BuildTest): > """Build core-image-minimal and test the results""" > I'd suggest this part of then change be split into a separate patch with an explanation of what is being fixed. It does appear to fix the database locked issues we were seeing which is great. Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#6044): https://lists.yoctoproject.org/g/toaster/message/6044 Mute This Topic: https://lists.yoctoproject.org/mt/102893708/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/toaster/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
