[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/4] iotests: add script_initialize
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/4] iotests: add script_initialize |
Date: |
Thu, 29 Aug 2019 14:43:20 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 8/29/19 2:27 PM, Philippe Mathieu-Daudé wrote:
> This restrict test 208 to the Linux platform, is this OK?
>
> The rest looks good.
I forgot about that, so good catch.
I don't know. We seem to already restrict a LOT of tests to the Linux
platform. Does this one actually work on other platforms?
Actually, I can't see any other supported oses/supported platforms calls
anywhere in iotests that don't just specify Linux or leave it the
default (...which is also linux.)
There isn't a way to engage the old-style python unittest framework
without implying Linux; you have to manually override it if so.
For new tests, MOST of them specified Linux in some way or another, as
you saw.
So either:
- 208 was an oversight, or
- Many tests are accidentally limiting to Linux and could be loosened.
Which is it? Dunno. Guess I'll look at the VM tests to see if I can
co-opt some of that... stay tuned?
--js
Re: [Qemu-devel] [PATCH v3 0/4] iotests: use python logging, no-reply, 2019/08/20
Re: [Qemu-devel] [PATCH v3 0/4] iotests: use python logging, John Snow, 2019/08/28