Re: [PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-16 Thread Vladimir Sementsov-Ogievskiy
16.12.2021 20:22, John Snow wrote: On Thu, Dec 16, 2021 at 4:51 AM Vladimir Sementsov-Ogievskiy mailto:vsement...@virtuozzo.com>> wrote: 15.12.2021 22:39, John Snow wrote: > This exception can be injected into any await statement. If we are > canceled via timeout, we want to clea

Re: [PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-16 Thread John Snow
On Thu, Dec 16, 2021 at 7:39 AM Beraldo Leal wrote: > On Wed, Dec 15, 2021 at 02:39:16PM -0500, John Snow wrote: > > This exception can be injected into any await statement. If we are > > canceled via timeout, we want to clear the pending execution record on > > our way out. > > > > Signed-off-by

Re: [PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-16 Thread John Snow
On Thu, Dec 16, 2021 at 4:51 AM Vladimir Sementsov-Ogievskiy < vsement...@virtuozzo.com> wrote: > 15.12.2021 22:39, John Snow wrote: > > This exception can be injected into any await statement. If we are > > canceled via timeout, we want to clear the pending execution record on > > our way out. >

Re: [PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-16 Thread Beraldo Leal
On Wed, Dec 15, 2021 at 02:39:16PM -0500, John Snow wrote: > This exception can be injected into any await statement. If we are > canceled via timeout, we want to clear the pending execution record on > our way out. > > Signed-off-by: John Snow > --- > python/qemu/aqmp/qmp_client.py | 8 ++--

Re: [PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-16 Thread Vladimir Sementsov-Ogievskiy
15.12.2021 22:39, John Snow wrote: This exception can be injected into any await statement. If we are canceled via timeout, we want to clear the pending execution record on our way out. Hmm, but there are more await statements in the file, shouldn't we care about them too ? Signed-off-by: J

[PATCH v2 02/25] python/aqmp: handle asyncio.TimeoutError on execute()

2021-12-15 Thread John Snow
This exception can be injected into any await statement. If we are canceled via timeout, we want to clear the pending execution record on our way out. Signed-off-by: John Snow --- python/qemu/aqmp/qmp_client.py | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/python/qe