On 12/19/18 9:53 PM, Eric Blake wrote:
> On 12/19/18 8:29 PM, John Snow wrote:
>> As laid out in the previous commit's message:
>>
>> ```
>> Several places in iotests deal with serializing objects into JSON
>> strings, but to add pretty-printing it seems desireable to localize
>
> s/desireable/desirable/
>
>> all of those cases.
>>
>> log() seems like a good candidate for that centralized behavior.
>> log() can already serialize json objects, but when it does so,
>> it assumes filters=[] operates on QMP objects, not strings.
>>
>> qmp_log currently operates by dumping outgoing and incoming QMP
>> objects into strings and filtering them assuming that filters=[]
>> are string filters.
>> ```
>>
>> Therefore:
>>
>> Change qmp_log to treat filters as if they're always qmp object filters,
>> then change the logging call to rely on log()'s ability to serialize QMP
>> objects, so we're not duplicating that effort.
>>
>> Add a qmp version of filter_testfiles and adjust the only caller using
>> it for qmp_log to use the qmp version.
>>
>> Signed-off-by: John Snow <js...@redhat.com>
>> Signed-off-by: John Snow <js...@redhat.com>
>
> Odd double S-o-B differing only by space.
I fixed my auto-signer! It has rudely detected my typo and decided that
it needed a fresh SOB.
>
>> ---
>> tests/qemu-iotests/206 | 4 ++--
>> tests/qemu-iotests/iotests.py | 24 +++++++++++++++++++++---
>> 2 files changed, 23 insertions(+), 5 deletions(-)
>>
>
> Reviewed-by: Eric Blake <ebl...@redhat.com>
>
Thanks!
