On Tue, 5 Nov 2024 at 13:23, Cédric Le Goater <c...@redhat.com> wrote:
>
> Since commit 786bc2255256, cached asset files are read-only. Adjust
> the QEMU command line for buildroot tests to reflect the new
> constraint on the flash drive.
>
> Fixes: f04cb2d00d5c ("tests/functional: Convert most Aspeed machine tests")
> Suggested-by: Thomas Huth <th...@redhat.com>
> Signed-off-by: Cédric Le Goater <c...@redhat.com>
> ---
> tests/functional/test_arm_aspeed.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/functional/test_arm_aspeed.py
> b/tests/functional/test_arm_aspeed.py
> index 9761fc06a454..19853161b267 100644
> --- a/tests/functional/test_arm_aspeed.py
> +++ b/tests/functional/test_arm_aspeed.py
> @@ -125,7 +125,7 @@ def test_arm_ast2500_romulus_openbmc_v2_9_0(self):
> def do_test_arm_aspeed_buildroot_start(self, image, cpu_id,
> pattern='Aspeed EVB'):
> self.require_netdev('user')
> self.vm.set_console()
> - self.vm.add_args('-drive', 'file=' + image + ',if=mtd,format=raw',
> + self.vm.add_args('-drive', 'file=' + image +
> ',if=mtd,format=raw,read-only=true',
> '-net', 'nic', '-net', 'user')
There are a couple of other places in this file where we add
a '-drive' option pointing to a file -- do they also need
read-only=true adding ?
thanks
-- PMM
