[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes |
Date: |
Tue, 22 Aug 2017 09:24:16 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
John Snow <address@hidden> writes:
> On 08/18/2017 10:26 AM, Lukáš Doktor wrote:
>> Hello guys,
>>
>> I'm reading the available python modules to exercise qemu and while reading
>> them
>> I fixed some issues that caught my attention. It usually starts with a simple
>> pylint/docstring fixes and slowly graduates to more controversial ones so I'm
>> open to suggestion to remove some of them.
>>
>> Kind regards,
>> Lukáš
>>
>> Changes in v2
>> - Squashed 2nd and 10th patches into 2nd one
>> - Use repr() in MonitorResponseError's description
>> - Improved commit message of the 6th patch
>> - Two tweaks to docstrings changed in the 6th patch
>> - Also updated qmp-shell to use new-style super calls (7th patch)
>> - Fixed the accidental rename of qmp `cmd_id` (kept the id->cmd_id patch)
>> - Changed the style of the style-fix in the 10th commit
>>
>> Changes in v3
>> - Don't use repr in the 5th patch in MonitorResponseError
>>
>> Changes in v4
>> - Use correct git base (remove unwanted commits)
>>
>> Changes in v5
>> - Avoid bool comparison
>> - Change report to return in one docstring
>> - Removed the unnecessary spaces around single-line docstring
>>
>> Changes in v6
>> - Bunch of docstring tweaks by Markus Armbruster
>> - Line break in <80 chars
>> - result dict => response dict
>> - Removed the "event_match" rename
>>
>
> Looks like all ten patches have an R-B despite changes; but it looks
> like nothing particularly major was changed anyway.
>
> Does this fall under Markus's jurisdiction?
>
> (Well, except for qtest.py which seemingly has double-extra-no
> maintainer...!)
qemu.py is about starting and controlling QEMU, commonly for testing
purposes. It's related to QMP only by virtue of using QMP for control
(well, what else could it use?); if that makes me maintainer, I'll soon
maintain basically all tests :)
As far as I can tell, qemu.py's main user is still qemu-iotests, via
qtest.py. Dan factored it out to make it available for
tests/migration/guestperf/.
Options for maintaining qemu.py and qtest.py:
* Maintain them with qemu-iotest
Currently mainained with the block layer core, by Kevin and Max.
- Keep it that way
- Appoint qemu-iotest maintainer(s).
* Maintain them separately, say as "Python qtest support", appoint
maintainer(s)
Dan appears to be a hot contender:
$ scripts/get_maintainer.pl -f --git-blame scripts/qtest.py
Fam Zheng <address@hidden> (authored lines:71/110=65%,commits:1/3=33%)
"Daniel P. Berrange" <address@hidden> (authored
lines:39/110=35%,commits:2/3=67%)
Max Reitz <address@hidden> (commits:2/3=67%)
Amit Shah <address@hidden> (commits:1/3=33%)
Stefan Hajnoczi <address@hidden> (commits:1/3=33%)
address@hidden (open list:All patches CC here)
$ scripts/get_maintainer.pl -f --git-blame scripts/qemu.py
"Daniel P. Berrange" <address@hidden> (authored
lines:217/229=95%,commits:2/4=50%)
Eduardo Habkost <address@hidden> (authored lines:12/229=5%,commits:4/4=100%)
Markus Armbruster <address@hidden> (commits:1/4=25%)
Max Reitz <address@hidden> (commits:1/4=25%)
Amit Shah <address@hidden> (commits:1/4=25%)
address@hidden (open list:All patches CC here)
Eduardo made the mistake^W^W^Wgraciously volunteered to maintain
scripts/qmp/qmp-shell, which is also used for testing. Perhaps he'd
be willing to maintain these guys as well.
* Do nothing
Hope "somebody" will take pity and merge patches. A common value of
"somebody" would be Paolo.
- Re: [Qemu-devel] [PATCH v6 03/10] qemu.py: Use iteritems rather than keys(), (continued)
- [Qemu-devel] [PATCH v6 04/10] qemu.py: Simplify QMP key-conversion, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 05/10] qemu.py: Use custom exceptions rather than Exception, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 06/10] qmp.py: Couple of pylint/style fixes, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 07/10] qmp.py: Use object-based class for QEMUMonitorProtocol, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 08/10] qmp.py: Avoid "has_key" usage, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 09/10] qmp.py: Avoid overriding a builtin object, Lukáš Doktor, 2017/08/18
- [Qemu-devel] [PATCH v6 10/10] qtest.py: Few pylint/style fixes, Lukáš Doktor, 2017/08/18
- Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes, John Snow, 2017/08/21
- Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes, Paolo Bonzini, 2017/08/22
- Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes, John Snow, 2017/08/22
- Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes, Eduardo Habkost, 2017/08/22
Re: [Qemu-devel] [PATCH v6 00/10] qemu.py: Pylint/style fixes, Eduardo Habkost, 2017/08/30