Browse Source

python: linter changes for pylint 3.x

New bleeding edge versions, new nits to iron out. This addresses the
'check-python-tox' optional GitLab test, while 'check-python-minreqs'
saw no regressions, since it's frozen on an older version of pylint.

Fixes:
qemu/machine/machine.py:345:52: E0606: Possibly using variable 'sock' before assignment (possibly-used-before-assignment)
qemu/utils/qemu_ga_client.py:168:4: R1711: Useless return at end of function or method (useless-return)

Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 20240626232230.408004-2-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
pull/286/head
John Snow 2 years ago
parent
commit
84e327e844
  1. 1
      python/qemu/machine/machine.py
  2. 2
      python/qemu/utils/qemu_ga_client.py

1
python/qemu/machine/machine.py

@ -335,6 +335,7 @@ class QEMUMachine:
def _pre_launch(self) -> None:
if self._qmp_set:
sock = None
if self._monitor_address is None:
self._sock_pair = socket.socketpair()
os.set_inheritable(self._sock_pair[0].fileno(), True)

2
python/qemu/utils/qemu_ga_client.py

@ -174,7 +174,7 @@ class QemuGuestAgentClient:
# On error exception will raise
except asyncio.TimeoutError:
# On success command will timed out
return
pass
def shutdown(self, mode: str = 'powerdown') -> None:
if mode not in ['powerdown', 'halt', 'reboot']:

Loading…
Cancel
Save