[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 5/5] Revert "python/qmp/protocol: add open_with_socket()"
From: |
John Snow |
Subject: |
[PULL 5/5] Revert "python/qmp/protocol: add open_with_socket()" |
Date: |
Wed, 31 May 2023 16:43:38 -0400 |
This reverts commit a3cfea92e2030926e00a2519d299384ea648e36e.
(It's being rolled back in favor of a different API, which brings the
in-tree and out-of-tree versions of qemu.qmp back in sync.)
Signed-off-by: John Snow <jsnow@redhat.com>
Message-id: 20230517163406.2593480-6-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
---
python/qemu/qmp/protocol.py | 24 +++++-------------------
1 file changed, 5 insertions(+), 19 deletions(-)
diff --git a/python/qemu/qmp/protocol.py b/python/qemu/qmp/protocol.py
index d534db4631..753182131f 100644
--- a/python/qemu/qmp/protocol.py
+++ b/python/qemu/qmp/protocol.py
@@ -297,19 +297,6 @@ async def start_server_and_accept(
await self.accept()
assert self.runstate == Runstate.RUNNING
- @upper_half
- @require(Runstate.IDLE)
- async def open_with_socket(self, sock: socket.socket) -> None:
- """
- Start connection with given socket.
-
- :param sock: A socket.
-
- :raise StateError: When the `Runstate` is not `IDLE`.
- """
- self._reader, self._writer = await asyncio.open_connection(sock=sock)
- self._set_state(Runstate.CONNECTING)
-
@upper_half
@require(Runstate.IDLE)
async def start_server(self, address: SocketAddrT,
@@ -357,12 +344,11 @@ async def accept(self) -> None:
protocol-level failure occurs while establishing a new
session, the wrapped error may also be an `QMPError`.
"""
- if not self._reader:
- if self._accepted is None:
- raise QMPError("Cannot call accept() before start_server().")
- await self._session_guard(
- self._do_accept(),
- 'Failed to establish connection')
+ if self._accepted is None:
+ raise QMPError("Cannot call accept() before start_server().")
+ await self._session_guard(
+ self._do_accept(),
+ 'Failed to establish connection')
await self._session_guard(
self._establish_session(),
'Failed to establish session')
--
2.40.1
- [PULL 0/5] Python patches, John Snow, 2023/05/31
- [PULL 2/5] python/qmp/legacy: allow using sockets for connect(), John Snow, 2023/05/31
- [PULL 1/5] python/qmp: allow sockets to be passed to connect(), John Snow, 2023/05/31
- [PULL 3/5] python/machine: use connect-based interface for existing sockets, John Snow, 2023/05/31
- [PULL 5/5] Revert "python/qmp/protocol: add open_with_socket()",
John Snow <=
- [PULL 4/5] python/qmp/legacy: remove open_with_socket() calls, John Snow, 2023/05/31
- Re: [PULL 0/5] Python patches, Richard Henderson, 2023/05/31