[PULL 00/22] Python patches

John Snow posted 22 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211101173006.656673-1-jsnow@redhat.com
Maintainers: Cleber Rosa <crosa@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>, Eduardo Habkost <ehabkost@redhat.com>, John Snow <jsnow@redhat.com>
There is a newer version of this series
python/qemu/aqmp/__init__.py              |  12 --
python/qemu/aqmp/legacy.py                | 138 ++++++++++++++++++++++
python/qemu/machine/machine.py            |  85 +++++++++----
python/tests/iotests-mypy.sh              |   4 +
python/tests/iotests-pylint.sh            |   4 +
scripts/simplebench/bench_block_job.py    |   3 +-
tests/qemu-iotests/040                    |   7 +-
tests/qemu-iotests/218                    |   2 +-
tests/qemu-iotests/255                    |   2 +-
tests/qemu-iotests/297                    | 103 +++++++---------
tests/qemu-iotests/300                    |  13 +-
tests/qemu-iotests/iotests.py             |  20 +++-
tests/qemu-iotests/linters.py             | 105 ++++++++++++++++
tests/qemu-iotests/mypy.ini               |  12 ++
tests/qemu-iotests/pylintrc               |  16 +++
tests/qemu-iotests/tests/mirror-top-perms |  17 ++-
16 files changed, 424 insertions(+), 119 deletions(-)
create mode 100644 python/qemu/aqmp/legacy.py
create mode 100755 python/tests/iotests-mypy.sh
create mode 100755 python/tests/iotests-pylint.sh
create mode 100644 tests/qemu-iotests/linters.py
create mode 100644 tests/qemu-iotests/mypy.ini
[PULL 00/22] Python patches
Posted by John Snow 2 years, 6 months ago
The following changes since commit af531756d25541a1b3b3d9a14e72e7fedd941a2e:

  Merge remote-tracking branch 'remotes/philmd/tags/renesas-20211030' into staging (2021-10-30 11:31:41 -0700)

are available in the Git repository at:

  https://gitlab.com/jsnow/qemu.git tags/python-pull-request

for you to fetch changes up to 76cd358671e6b8e7c435ec65b1c44200254514a9:

  python, iotests: replace qmp with aqmp (2021-11-01 11:54:59 -0400)

----------------------------------------------------------------
Pull request

----------------------------------------------------------------

John Snow (22):
  iotests/297: Move pylint config into pylintrc
  iotests/297: Split mypy configuration out into mypy.ini
  iotests/297: Add get_files() function
  iotests/297: Create main() function
  iotests/297: Don't rely on distro-specific linter binaries
  iotests/297: Split run_linters apart into run_pylint and run_mypy
  iotests/297: refactor run_[mypy|pylint] as generic execution shim
  iotests/297: Change run_linter() to raise an exception on failure
  iotests/297: update tool availability checks
  iotests/297: split test into sub-cases
  iotests: split linters.py out from 297
  iotests/linters: Add entry point for linting via Python CI
  iotests/linters: Add workaround for mypy bug #9852
  python: Add iotest linters to test suite
  python/machine: remove has_quit argument
  python/machine: Handle QMP errors on close more meticulously
  python/aqmp: Remove scary message
  iotests: Accommodate async QMP Exception classes
  iotests: Conditionally silence certain AQMP errors
  iotests/300: avoid abnormal shutdown race condition
  python/aqmp: Create sync QMP wrapper for iotests
  python, iotests: replace qmp with aqmp

 python/qemu/aqmp/__init__.py              |  12 --
 python/qemu/aqmp/legacy.py                | 138 ++++++++++++++++++++++
 python/qemu/machine/machine.py            |  85 +++++++++----
 python/tests/iotests-mypy.sh              |   4 +
 python/tests/iotests-pylint.sh            |   4 +
 scripts/simplebench/bench_block_job.py    |   3 +-
 tests/qemu-iotests/040                    |   7 +-
 tests/qemu-iotests/218                    |   2 +-
 tests/qemu-iotests/255                    |   2 +-
 tests/qemu-iotests/297                    | 103 +++++++---------
 tests/qemu-iotests/300                    |  13 +-
 tests/qemu-iotests/iotests.py             |  20 +++-
 tests/qemu-iotests/linters.py             | 105 ++++++++++++++++
 tests/qemu-iotests/mypy.ini               |  12 ++
 tests/qemu-iotests/pylintrc               |  16 +++
 tests/qemu-iotests/tests/mirror-top-perms |  17 ++-
 16 files changed, 424 insertions(+), 119 deletions(-)
 create mode 100644 python/qemu/aqmp/legacy.py
 create mode 100755 python/tests/iotests-mypy.sh
 create mode 100755 python/tests/iotests-pylint.sh
 create mode 100644 tests/qemu-iotests/linters.py
 create mode 100644 tests/qemu-iotests/mypy.ini

-- 
2.31.1



Re: [PULL 00/22] Python patches
Posted by Richard Henderson 2 years, 6 months ago
On 11/1/21 1:29 PM, John Snow wrote:
> The following changes since commit af531756d25541a1b3b3d9a14e72e7fedd941a2e:
> 
>    Merge remote-tracking branch 'remotes/philmd/tags/renesas-20211030' into staging (2021-10-30 11:31:41 -0700)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/jsnow/qemu.git tags/python-pull-request
> 
> for you to fetch changes up to 76cd358671e6b8e7c435ec65b1c44200254514a9:
> 
>    python, iotests: replace qmp with aqmp (2021-11-01 11:54:59 -0400)
> 
> ----------------------------------------------------------------
> Pull request
> 
> ----------------------------------------------------------------
> 
> John Snow (22):
>    iotests/297: Move pylint config into pylintrc
>    iotests/297: Split mypy configuration out into mypy.ini
>    iotests/297: Add get_files() function
>    iotests/297: Create main() function
>    iotests/297: Don't rely on distro-specific linter binaries
>    iotests/297: Split run_linters apart into run_pylint and run_mypy
>    iotests/297: refactor run_[mypy|pylint] as generic execution shim
>    iotests/297: Change run_linter() to raise an exception on failure
>    iotests/297: update tool availability checks
>    iotests/297: split test into sub-cases
>    iotests: split linters.py out from 297
>    iotests/linters: Add entry point for linting via Python CI
>    iotests/linters: Add workaround for mypy bug #9852
>    python: Add iotest linters to test suite
>    python/machine: remove has_quit argument
>    python/machine: Handle QMP errors on close more meticulously
>    python/aqmp: Remove scary message
>    iotests: Accommodate async QMP Exception classes
>    iotests: Conditionally silence certain AQMP errors
>    iotests/300: avoid abnormal shutdown race condition
>    python/aqmp: Create sync QMP wrapper for iotests
>    python, iotests: replace qmp with aqmp
> 
>   python/qemu/aqmp/__init__.py              |  12 --
>   python/qemu/aqmp/legacy.py                | 138 ++++++++++++++++++++++
>   python/qemu/machine/machine.py            |  85 +++++++++----
>   python/tests/iotests-mypy.sh              |   4 +
>   python/tests/iotests-pylint.sh            |   4 +
>   scripts/simplebench/bench_block_job.py    |   3 +-
>   tests/qemu-iotests/040                    |   7 +-
>   tests/qemu-iotests/218                    |   2 +-
>   tests/qemu-iotests/255                    |   2 +-
>   tests/qemu-iotests/297                    | 103 +++++++---------
>   tests/qemu-iotests/300                    |  13 +-
>   tests/qemu-iotests/iotests.py             |  20 +++-
>   tests/qemu-iotests/linters.py             | 105 ++++++++++++++++
>   tests/qemu-iotests/mypy.ini               |  12 ++
>   tests/qemu-iotests/pylintrc               |  16 +++
>   tests/qemu-iotests/tests/mirror-top-perms |  17 ++-
>   16 files changed, 424 insertions(+), 119 deletions(-)
>   create mode 100644 python/qemu/aqmp/legacy.py
>   create mode 100755 python/tests/iotests-mypy.sh
>   create mode 100755 python/tests/iotests-pylint.sh
>   create mode 100644 tests/qemu-iotests/linters.py
>   create mode 100644 tests/qemu-iotests/mypy.ini

Applied, thanks.

r~