Am 31.01.2019 um 03:03 hat John Snow geschrieben:
> On 1/30/19 12:27 PM, Kevin Wolf wrote:
> > Am 14.01.2019 um 17:25 hat Eric Blake geschrieben:
> >> From: John Snow <[email protected]>
> >>
> >> New interface, new smoke test.
> >>
> >> Signed-off-by: John Snow <[email protected]>
> >> Reviewed-by: Vladimir Sementsov-Ogievskiy <[email protected]>
> >> Reviewed-by: Eric Blake <[email protected]>
> >> Message-Id: <[email protected]>
> >> [eblake: fix last-minute change to echo text]
> >> Signed-off-by: Eric Blake <[email protected]>
> >
> > This test fails for me (with Python 3) because I get a different dict
> > order.
> >
> > Probably related to the OrderedDict that qmp_log() uses, so log() uses
> > sort_keys=False and the unordered dicts that are contained in the outer
> > OrderedDict stay unordered.
> >
> > Kevin
> >
>
> OK, the problem is that ordered_kwargs() is not recursively ordering
> those kwargs beneath list entries, so transactions are avoiding the sort.
>
> Patch coming up, thank you for the report and my apologies for breaking
> iotests so often this cycle.
>
> (While I'm here, how do you configure iotests to use your python3
> binary? I tried at configure time but that breaks the build for me with
> some "magic number" errors. I could toy with it after by editing
> common.env, but should the build work with python3?)
Yes, just at configure time:
./configure --target-list=x86_64-softmmu --python=/usr/bin/python3
I'm building and running tests in-tree, if that makes a difference.
But maybe it's actually not so bad if at least one of us still tests
with Python 2 as long as we haven't officially removed support for
that...
Kevin
