Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Vladimir Sementsov-Ogievskiy
22.01.2021 16:34, Kevin Wolf wrote: Am 22.01.2021 um 14:16 hat Vladimir Sementsov-Ogievskiy geschrieben: 22.01.2021 15:45, Kevin Wolf wrote: Am 22.01.2021 um 12:57 hat Vladimir Sementsov-Ogievskiy geschrieben: 22.01.2021 14:48, Kevin Wolf wrote: Am 16.01.2021 um 14:44 hat Vladimir

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Kevin Wolf
Am 22.01.2021 um 14:16 hat Vladimir Sementsov-Ogievskiy geschrieben: > 22.01.2021 15:45, Kevin Wolf wrote: > > Am 22.01.2021 um 12:57 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > 22.01.2021 14:48, Kevin Wolf wrote: > > > > Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben:

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Vladimir Sementsov-Ogievskiy
22.01.2021 15:45, Kevin Wolf wrote: Am 22.01.2021 um 12:57 hat Vladimir Sementsov-Ogievskiy geschrieben: 22.01.2021 14:48, Kevin Wolf wrote: Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: +def add_group_file(self, fname: str) -> None: +with open(fname) as f:

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Kevin Wolf
Am 22.01.2021 um 12:57 hat Vladimir Sementsov-Ogievskiy geschrieben: > 22.01.2021 14:48, Kevin Wolf wrote: > > Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > +def add_group_file(self, fname: str) -> None: > > > +with open(fname) as f: > > > +for

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Vladimir Sementsov-Ogievskiy
22.01.2021 14:49, Kevin Wolf wrote: Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: +if 'disabled' not in groups and 'disabled' not in exclude_groups: +# Don't want to modify function argument, so create new list. +exclude_groups =

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Vladimir Sementsov-Ogievskiy
22.01.2021 14:48, Kevin Wolf wrote: Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: Add python script with new logic of searching for tests: Current ./check behavior: - tests are named [0-9][0-9][0-9] - tests must be registered in group file (even if test doesn't

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Kevin Wolf
Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: > +if 'disabled' not in groups and 'disabled' not in exclude_groups: > +# Don't want to modify function argument, so create new list. > +exclude_groups = exclude_groups + ['disabled'] Oops, forgot

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-22 Thread Kevin Wolf
Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben: > Add python script with new logic of searching for tests: > > Current ./check behavior: > - tests are named [0-9][0-9][0-9] > - tests must be registered in group file (even if test doesn't belong >to any group, like 142)

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-21 Thread Vladimir Sementsov-Ogievskiy
21.01.2021 19:21, Eric Blake wrote: On 1/16/21 7:44 AM, Vladimir Sementsov-Ogievskiy wrote: Add python script with new logic of searching for tests: The file findtests.py is self-executable and may be used for debugging purposes. Signed-off-by: Vladimir Sementsov-Ogievskiy --- diff

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-21 Thread Eric Blake
On 1/16/21 7:44 AM, Vladimir Sementsov-Ogievskiy wrote: > Add python script with new logic of searching for tests: > > Current ./check behavior: > - tests are named [0-9][0-9][0-9] > - tests must be registered in group file (even if test doesn't belong >to any group, like 142) > > Behavior

Re: [PATCH v7 07/11] iotests: add findtests.py

2021-01-21 Thread Eric Blake
On 1/16/21 7:44 AM, Vladimir Sementsov-Ogievskiy wrote: > Add python script with new logic of searching for tests: > > The file findtests.py is self-executable and may be used for debugging > purposes. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > diff --git

[PATCH v7 07/11] iotests: add findtests.py

2021-01-16 Thread Vladimir Sementsov-Ogievskiy
Add python script with new logic of searching for tests: Current ./check behavior: - tests are named [0-9][0-9][0-9] - tests must be registered in group file (even if test doesn't belong to any group, like 142) Behavior of findtests.py: - group file is dropped - tests are all files in