Re: [Qemu-ppc] [PATCH v2 0/6] tests/acceptance: Add tests for the PReP/40p machine

[Philippe Mathieu-Daudé]

On 9/16/19 11:52 AM, Alex Bennée wrote:
> 
> Philippe Mathieu-Daudé <address@hidden> writes:
> 
>> Hi David,
>>
>> On 9/16/19 2:42 AM, David Gibson wrote:
>>> On Sun, Sep 15, 2019 at 11:19:34PM +0200, Philippe Mathieu-Daudé wrote:
>>>> Quick tests worth to avoid regressions with the 40p machine.
>>>> idea from the "Maintainers, please tell us how to boot your machines"
>>>> thread:
>>>> https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg04177.html
>>>>
>>>> v2: Split Travis job, added Hervé R-b tag
>>>> v1: https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg05896.html
>>>>
>>>> Regards,
>>>>
>>>> Phil.
>>>
>>> I'm guessing you're expecting these to go in via the testing tree, in
>>> which case
>>>
>>> Acked-by: David Gibson <address@hidden>
>>
>> Thanks, appreciated :)
>>
>>> Or do you want me to take them via the ppc tree?
>>
>> I think the 'testing tree' should focus on the CI/testing
>> infrastructure, while each subsystem maintainers should care about the
>> tests covering their subsystem (the testing tree maintainers might not
>> have the required knowledge to be sure a test is correctly implemented).
>>
>> In this particular case I assume you don't have much knowledge of that
>> PPC machine, which is a hobbyist one, but since you are the PPC
>> maintainer, I'd rather see this going via your tree :)
>>
>> Alex/Cleber/Eduardo, any comment on this position?
> 
> Once we have a .travis.yml I'm happy with it can go in via another tree
> no problem. See other thread....

Good :)

David can take patches 1-5 (I tagged patch 6 as RFC but messed something
with git-publish and lost it when I sent this series).

Thanks!

>>>> Philippe Mathieu-Daudé (6):
>>>>   tests/acceptance: Add test that runs NetBSD 4.0 installer on PRep/40p
>>>>   tests/acceptance: Test Open Firmware on the PReP/40p
>>>>   tests/acceptance: Test OpenBIOS on the PReP/40p
>>>>   tests/acceptance: Test Sandalfoot initrd on the PReP/40p
>>>>   .travis.yml: Let the avocado job run the 40p tests
>>>>   .travis.yml: Split enterprise vs. hobbyist acceptance test job
>>>>
>>>>  .travis.yml                      |  18 +++-
>>>>  MAINTAINERS                      |   1 +
>>>>  tests/acceptance/ppc_prep_40p.py | 150 +++++++++++++++++++++++++++++++
>>>>  3 files changed, 167 insertions(+), 2 deletions(-)
>>>>  create mode 100644 tests/acceptance/ppc_prep_40p.py
>>>>
>>>
> 
> 
> --
> Alex Bennée
>