[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platfo
From: |
Max Reitz |
Subject: |
Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms |
Date: |
Mon, 7 Oct 2019 18:55:19 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 07.10.19 18:28, Thomas Huth wrote:
> On 07/10/2019 15.11, Thomas Huth wrote:
>> On 07/10/2019 14.52, Max Reitz wrote:
>>> On 07.10.19 14:16, Thomas Huth wrote:
>>>> On 04/10/2019 14.44, Max Reitz wrote:
>>>>> On 04.10.19 12:19, Kevin Wolf wrote:
>>>>>> Am 02.10.2019 um 19:47 hat Max Reitz geschrieben:
>>>>>>> On 02.10.19 18:44, Kevin Wolf wrote:
>>>>>>>> Am 02.10.2019 um 13:57 hat Max Reitz geschrieben:
>>>>>>>>> It usually worked fine for me because it’s rather rare that non-block
>>>>>>>>> patches broke the iotests.
>>>>>>>>
>>>>>>>> I disagree. It happened all the time that someone else broke the
>>>>>>>> iotests
>>>>>>>> in master and we needed to fix it up.
>>>>>>>
>>>>>>> OK.
>>>>>>>
>>>>>>> (In my experience, it’s still mostly block patches, only that they tend
>>>>>>> to go through non-block trees.)
>>>>>>
>>>>>> Fair enough, it's usually code that touches block code. I assumed "block
>>>>>> patches" to mean patches that go through one of the block trees and for
>>>>>> which iotests are run before sending a pull request.
>>>>>>
>>>>>> In the end, I don't care what code these patches touched. I do an
>>>>>> innocent git pull, and when I finally see that it's master that breaks
>>>>>> iotests and not my patches on top of it, I'm annoyed.
>>>>>
>>>>> Hm. Part of my point was that this still happens all the time.
>>>>>
>>>>> Which is why I’d prefer more tests to run in CI than a handful of not
>>>>> very useful ones in make check.
>>>>
>>>> Ok, so let's try to add some more useful test to the "auto" group. Kevin
>>>> mentioned 030, 040 and 041, and I think it should be ok to enable them
>>>> (IIRC the only issue was that they run a little bit longer, but if they
>>>> are very useful, we should include them anyway).
>>>
>>> I agree on those. (Maybe not 040, but definitely 030 and 041.)
>>>
>>> Maybe one of the issues was the “path too long” thing for Unix sockets?
>>
>> Ah, right. I've applied John's "remove 'linux' from default" patch and
>> added the three iotests to the "auto" group, and indeed, they fail now
>> on cirrus-ci due to the "path too long" socket problem. "We" (royal we,
>> I guess) should likely fix that first...
I’m looking into that, by the way. I hope a SOCK_DIR that defaults to
$(mktemp -d) will suffice...
> FWIW, 041 also fails on macOS on Travis (which does not have the "path
> too long" issue):
>
> https://travis-ci.com/huth/qemu/jobs/242942716#L8415
>
> ... so we might need to declare this as "linux only" again after John's
> patch gets merged.
:/
Either that, or let make check skip the iotests generally on non-Linux
systems.
Max
signature.asc
Description: OpenPGP digital signature
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, (continued)
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Max Reitz, 2019/10/04
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Peter Maydell, 2019/10/04
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Max Reitz, 2019/10/04
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Peter Maydell, 2019/10/04
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Max Reitz, 2019/10/04
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Thomas Huth, 2019/10/07
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Peter Maydell, 2019/10/07
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Max Reitz, 2019/10/07
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Thomas Huth, 2019/10/07
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Thomas Huth, 2019/10/07
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms,
Max Reitz <=
- Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, Thomas Huth, 2019/10/02
Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms, John Snow, 2019/10/02