Re: [PATCH 2/4] CI: add smoke-tests for switch to dracut in isar-image-ci

[Zhihang Wei <wzh@ilbers.de>]

On 11/1/25 12:13, 'Cedric Hombourger' via isar-users wrote:
> Use IMAGE_PREINSTALL to install dracut in the isar-image-ci
> image and verify that it builds and boots. We also check that
> a dracut-shutdown service is active (as an indication that
> the system was booted using a dracut initrd). A single test
> is provided as well as cross-{machine,distro} tests.
>
> Signed-off-by: Cedric Hombourger <cedric.hombourger@siemens.com>
> ---
>   testsuite/citest.py | 58 +++++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 58 insertions(+)
>
> diff --git a/testsuite/citest.py b/testsuite/citest.py
> index a1214e9c..23900a6c 100755
> --- a/testsuite/citest.py
> +++ b/testsuite/citest.py
> @@ -278,6 +278,64 @@ class KernelTests(CIBaseTest):
>           self.perform_build_test(targets, image_install=' '.join(modules), lines=lines)
>   
>   
> +class InitRdBaseTest(CIBaseTest):
> +    """
> +    Helper class for InitRd build tests
> +    """
> +
> +    DRACUT_CONF = [
> +        "IMAGE_PREINSTALL:append = ' dracut'",
> +        "IMAGE_INSTALL:remove = 'enable-fsck'",
> +    ]
> +
> +    def dracut_in_image(self, targets):
> +        machine = 'qemuamd64'
> +        distro = 'bookworm'
> +        image = 'isar-image-ci'
> +        self.init()
> +        self.perform_build_test(targets, lines=InitRdBaseTest.DRACUT_CONF)
> +
> +        for target in targets:
> +            config = target.split(':')
> +            machine = config[1].split('-')[0]
> +            distro = config[1].split('-')[1]
> +            image = config[2]
> +
> +            # systems booted via a dracut initrd have a dracut-shutdown service
> +            # check its presence
> +            self.vm_start(
> +                machine.removeprefix('qemu'), distro, image=image,
> +                cmd="systemctl is-active dracut-shutdown"
> +            )
> +
> +
I would suggest to move the definition of class InitRdBaseTest(CIBaseTest)
to cibase.py. In citest.py there should only be definition of tests that we
can directly run.
> +class InitRdTest(InitRdBaseTest):
> +    """
> +    Tests for initrd generation
> +    :avocado: tags=initrd,single,full
> +    """
> +
> +    def test_dracut_in_image(self):
> +        """Test switch to dracut in an image recipe."""
> +        self.dracut_in_image(['mc:qemuamd64-bookworm:isar-image-ci'])
> +
> +
> +class InitRdCrossTests(InitRdBaseTest):
> +    """
> +    Tests for initrd generation
> +    :avocado: tags=initrd,full
> +    """
> +
> +    def test_dracut_in_image(self):
> +        """Test switch to dracut in an image recipe."""
> +        self.dracut_in_image([
> +            'mc:qemuamd64-bookworm:isar-image-ci',
> +            'mc:qemuamd64-trixie:isar-image-ci',
> +            'mc:qemuarm64-bookworm:isar-image-ci',
> +            'mc:qemuarm64-trixie:isar-image-ci'
> +        ])
> +
> +
>   class WicTest(CIBaseTest):
>   
>       """
Thanks for adding new test cases. I have tested the v7 dracut patch set
against these test cases and others in current CI, didn't show any error.
We'll continue with this patch set after dracut can be applied.

Best regards,
Zhihang

-- 
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/bda24ac5-21de-4881-bc64-704a58147f6b%40ilbers.de.