tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

[PATCH 0/3] tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

Thomas Huth posted 3 patches 1 year ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230421110345.1294131-1-thuth@redhat.com

Maintainers: Cleber Rosa <crosa@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Beraldo Leal <bleal@redhat.com>, Aurelien Jarno <aurelien@aurel32.net>, "Cédric Le Goater" <clg@kaod.org>, Peter Maydell <peter.maydell@linaro.org>, Andrew Jeffery <andrew@aj.id.au>, Joel Stanley <joel@jms.id.au>

MAINTAINERS                            |  2 +-
tests/avocado/avocado_qemu/__init__.py |  8 +++++++
tests/avocado/linux_ssh_mips_malta.py  |  8 -------
tests/avocado/machine_aspeed.py        | 31 +++++++++++++++-----------
4 files changed, 27 insertions(+), 22 deletions(-)

Expand all Fold all

[PATCH 0/3] tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

Posted by Thomas Huth 1 year ago

Fix the broken ast2500_evb_sdk and ast2600_evb_sdk avocado tests.
See the patch description of the second patch for details.
Also add the test to the MAINTAINERS file (third patch).

Thomas Huth (3):
  tests/avocado: Make ssh_command_output_contains() globally available
  tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests
  MAINTAINERS: Cover tests/avocado/machine_aspeed.py

 MAINTAINERS                            |  2 +-
 tests/avocado/avocado_qemu/__init__.py |  8 +++++++
 tests/avocado/linux_ssh_mips_malta.py  |  8 -------
 tests/avocado/machine_aspeed.py        | 31 +++++++++++++++-----------
 4 files changed, 27 insertions(+), 22 deletions(-)

-- 
2.31.1

Re: [PATCH 0/3] tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

Posted by Alex Bennée 1 year ago

Thomas Huth <thuth@redhat.com> writes:

> Fix the broken ast2500_evb_sdk and ast2600_evb_sdk avocado tests.
> See the patch description of the second patch for details.
> Also add the test to the MAINTAINERS file (third patch).
>
> Thomas Huth (3):
>   tests/avocado: Make ssh_command_output_contains() globally available
>   tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests
>   MAINTAINERS: Cover tests/avocado/machine_aspeed.py
>
>  MAINTAINERS                            |  2 +-
>  tests/avocado/avocado_qemu/__init__.py |  8 +++++++
>  tests/avocado/linux_ssh_mips_malta.py  |  8 -------
>  tests/avocado/machine_aspeed.py        | 31 +++++++++++++++-----------
>  4 files changed, 27 insertions(+), 22 deletions(-)

Queued to testing/next, thanks.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH 0/3] tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

Posted by Thomas Huth 1 year ago

On 21/04/2023 15.16, Alex Bennée wrote:
> 
> Thomas Huth <thuth@redhat.com> writes:
> 
>> Fix the broken ast2500_evb_sdk and ast2600_evb_sdk avocado tests.
>> See the patch description of the second patch for details.
>> Also add the test to the MAINTAINERS file (third patch).
>>
>> Thomas Huth (3):
>>    tests/avocado: Make ssh_command_output_contains() globally available
>>    tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests
>>    MAINTAINERS: Cover tests/avocado/machine_aspeed.py
>>
>>   MAINTAINERS                            |  2 +-
>>   tests/avocado/avocado_qemu/__init__.py |  8 +++++++
>>   tests/avocado/linux_ssh_mips_malta.py  |  8 -------
>>   tests/avocado/machine_aspeed.py        | 31 +++++++++++++++-----------
>>   4 files changed, 27 insertions(+), 22 deletions(-)
> 
> Queued to testing/next, thanks.

Thanks, but could you please remove that "self.log.info('going to starrt 
*******')" line in the second patch? That was a debugging left-over...

  Thomas

Re: [PATCH 0/3] tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests

Posted by Alex Bennée 1 year ago

Thomas Huth <thuth@redhat.com> writes:

> On 21/04/2023 15.16, Alex Bennée wrote:
>> Thomas Huth <thuth@redhat.com> writes:
>> 
>>> Fix the broken ast2500_evb_sdk and ast2600_evb_sdk avocado tests.
>>> See the patch description of the second patch for details.
>>> Also add the test to the MAINTAINERS file (third patch).
>>>
>>> Thomas Huth (3):
>>>    tests/avocado: Make ssh_command_output_contains() globally available
>>>    tests/avocado/machine_aspeed: Fix the broken ast2[56]00_evb_sdk tests
>>>    MAINTAINERS: Cover tests/avocado/machine_aspeed.py
>>>
>>>   MAINTAINERS                            |  2 +-
>>>   tests/avocado/avocado_qemu/__init__.py |  8 +++++++
>>>   tests/avocado/linux_ssh_mips_malta.py  |  8 -------
>>>   tests/avocado/machine_aspeed.py        | 31 +++++++++++++++-----------
>>>   4 files changed, 27 insertions(+), 22 deletions(-)
>> Queued to testing/next, thanks.
>
> Thanks, but could you please remove that "self.log.info('going to
> starrt *******')" line in the second patch? That was a debugging
> left-over...

no problem, done.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro