On 8/29/19 8:43 PM, John Snow wrote: > 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.
Then let's use: iotests.script_initialize(supported_oses=[]) and call it a day? > > 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. This can stay in your TODO for after this painful series. > 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 >