Re: [PATCH v7 00/15] iotests: use vm.cmd()

2023-10-12 Thread John Snow
On Thu, Oct 12, 2023 at 1:52 PM Vladimir Sementsov-Ogievskiy wrote: > > On 12.10.23 01:04, John Snow wrote: > > On Fri, Oct 6, 2023 at 11:41 AM Vladimir Sementsov-Ogievskiy > > wrote: > >> > >> Hi all! > >> > >> Let's get rid of pattern > >> > >> result = self.vm.qmp(...) > >>

Re: [PATCH v7 00/15] iotests: use vm.cmd()

2023-10-12 Thread Vladimir Sementsov-Ogievskiy
On 12.10.23 01:04, John Snow wrote: On Fri, Oct 6, 2023 at 11:41 AM Vladimir Sementsov-Ogievskiy wrote: Hi all! Let's get rid of pattern result = self.vm.qmp(...) self.assert_qmp(result, 'return', {}) And switch to just self.vm.cmd(...) v7: add r-bs and small

Re: [PATCH v7 00/15] iotests: use vm.cmd()

2023-10-11 Thread John Snow
On Fri, Oct 6, 2023 at 11:41 AM Vladimir Sementsov-Ogievskiy wrote: > > Hi all! > > Let's get rid of pattern > > result = self.vm.qmp(...) > self.assert_qmp(result, 'return', {}) > > And switch to just > > self.vm.cmd(...) > > v7: add r-bs and small wording/grammar fixes by Eric >

[PATCH v7 00/15] iotests: use vm.cmd()

2023-10-06 Thread Vladimir Sementsov-Ogievskiy
Hi all! Let's get rid of pattern result = self.vm.qmp(...) self.assert_qmp(result, 'return', {}) And switch to just self.vm.cmd(...) v7: add r-bs and small wording/grammar fixes by Eric 05: handle missed tests/avocado/machine_aspeed.py, keep r-bs 10: patch renamed: