[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 7/9] iotests.py: Filter filename in any string v
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH 7/9] iotests.py: Filter filename in any string value |
Date: |
Tue, 29 Jan 2019 16:36:25 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 1/23/19 9:46 AM, Max Reitz wrote:
> filter_qmp_testfiles() currently filters the filename only for specific
> keys. However, there are more keys that take filenames (such as
> block-commit's @top and @base, or ssh's @path), and it does not make
> sense to list them all here. "$TEST_DIR/$PID-" should have enough
> entropy not to appear anywhere randomly.
>
> Signed-off-by: Max Reitz <address@hidden>
There might be a more aggressive fix we can apply if we don't care about
the keys, but maybe some other filters do still care about the keys.
Ah, either way, this is obviously fine.
Reviewed-by: John Snow <address@hidden>
> ---
> tests/qemu-iotests/iotests.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
> index 29e4e3306f..b6cb73e108 100644
> --- a/tests/qemu-iotests/iotests.py
> +++ b/tests/qemu-iotests/iotests.py
> @@ -274,7 +274,7 @@ def filter_testfiles(msg):
>
> def filter_qmp_testfiles(qmsg):
> def _filter(key, value):
> - if key == 'filename' or key == 'backing-file':
> + if is_str(value):
> return filter_testfiles(value)
> return value
> return filter_qmp(qmsg, _filter)
>
--
—js
- Re: [Qemu-devel] [PATCH 5/9] iotests: Fix 237 for Python 2.x, (continued)
- [Qemu-devel] [PATCH 4/9] iotests: Fix 207 to use QMP filters for qmp_log, Max Reitz, 2019/01/23
- [Qemu-devel] [PATCH 3/9] iotests: Fix 232 for LUKS, Max Reitz, 2019/01/23
- [Qemu-devel] [PATCH 6/9] iotests.py: Add is_str(), Max Reitz, 2019/01/23
- [Qemu-devel] [PATCH 8/9] iotests: Filter SSH paths, Max Reitz, 2019/01/23
- [Qemu-devel] [PATCH 7/9] iotests.py: Filter filename in any string value, Max Reitz, 2019/01/23
- Re: [Qemu-devel] [PATCH 7/9] iotests.py: Filter filename in any string value,
John Snow <=
- [Qemu-devel] [PATCH 9/9] iotests.py: s/_/-/g on keys in qmp_log(), Max Reitz, 2019/01/23