Re: [PATCH v3] tests/functional: Convert most Aspeed machine tests

[Thomas Huth]

On 21/10/2024 11.31, Cédric Le Goater wrote:
> Hello Thomas,
>
>
> On 10/21/24 10:53, Thomas Huth wrote:
> > On 18/10/2024 09.17, Cédric Le Goater wrote:
> > > This is a simple conversion of the tests with some cleanups and
> > > adjustments to match the new test framework. Replace the zephyr image
> > > MD5 hashes with SHA256 hashes while at it.
> > >
> > > The SDK tests depend on a ssh class from avocado.utils which is
> > > difficult to replace. To be addressed separately.
> > >
> > > Signed-off-by: Cédric Le Goater <clg@redhat.com>
> > > ---
> >
> >   Hi Cédric,
> >
> > I'm seeing errors when I try to run "make check-functional" with this 
> > patch multiple times, e.g.:
> >
> >    File "/home/thuth/devel/qemu/tests/functional/test_arm_aspeed.py", line 
> > 111, in test_arm_ast2400_palmetto_openbmc_v2_9_0
> >      image_path = self.ASSET_PALMETTO_FLASH.fetch()
> >                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> >    File "/home/thuth/devel/qemu/tests/functional/qemu_test/asset.py", line 
> > 104, in fetch
> >      raise Exception("Asset cache is invalid and downloads disabled")
> > Exception: Asset cache is invalid and downloads disabled
> >
> > Could it be that the tests alter the file in the download cache?
>
> yes. the flash image is updated by Linux. I don't reproduce the issue
> on my build though. Are you using special options ?

No, but I rebased my staging branch to the current master shortly before ... 
maybe something else has just been merged into the current version? Have you 
tried rebasing your patch to the current master branch already?

 Thomas