[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/2] iotests: add another bash sleep command to 247
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH v2 1/2] iotests: add another bash sleep command to 247 |
Date: |
Wed, 25 Nov 2020 10:35:10 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
* Andrey Shinkevich (andrey.shinkevich@virtuozzo.com) wrote:
> On 24.11.2020 13:04, Vladimir Sementsov-Ogievskiy wrote:
> > 23.11.2020 18:44, Andrey Shinkevich wrote:
> > > This patch paves the way for the one that follows. The following patch
> > > makes the QMP monitor to read up to 4K from stdin at once. That results
> > > in running the bash 'sleep' command before the _qemu_proc_exec() starts
> >
> > But how? Before _qemu_proc_exec() starts, qemu monitor is not runnning,
> > and its new behavior can't influence..
> >
>
> I am not a bash expert to explain 'how' but this workaround works. It's just
> a test. Maybe other colleagues can say.
> > If bash subshell work in unpredictable way, may be better is refactor test
> > to send commands one by one with help of _send_qemu_cmd. Then sleep will
> > be natively executed between sending commands.
> >
>
> Or maybe write a similar test case in Python if Kevin agrees.
Yeh I don't believe the 'before the _qemu_proc_exec' starts- it's all
happening horribly in parallel with the subshell - the timing is
probably fragile as hell.
THe test needs to wait for output and then quit.
Dave
> > > in subshell. Another 'sleep' command with an unobtrusive 'query-status'
> > > plays as a workaround.
> > >
> > > Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com>
> > > ---
> > > Â tests/qemu-iotests/247Â Â Â Â | 2 ++
> > > Â tests/qemu-iotests/247.out | 1 +
> > > Â 2 files changed, 3 insertions(+)
> > >
> > > diff --git a/tests/qemu-iotests/247 b/tests/qemu-iotests/247
> > > index 87e37b3..7d316ec 100755
> > > --- a/tests/qemu-iotests/247
> > > +++ b/tests/qemu-iotests/247
> > > @@ -59,6 +59,8 @@ TEST_IMG="$TEST_IMG.4" _make_test_img $size
> > > Â {"execute":"block-commit",
> > > Â Â "arguments":{"device":"format-4", "top-node": "format-2",
> > > "base-node":"format-0", "job-id":"job0"}}
> > > Â EOF
> > > +sleep 1
> > > +echo '{"execute":"query-status"}'
> > > Â if [ "${VALGRIND_QEMU}" == "y" ]; then
> > > Â Â Â Â Â sleep 10
> > > Â else
> > > diff --git a/tests/qemu-iotests/247.out b/tests/qemu-iotests/247.out
> > > index e909e83..13d9547 100644
> > > --- a/tests/qemu-iotests/247.out
> > > +++ b/tests/qemu-iotests/247.out
> > > @@ -17,6 +17,7 @@ QMP_VERSION
> > > Â {"timestamp": {"seconds":Â TIMESTAMP, "microseconds":Â TIMESTAMP},
> > > "event": "BLOCK_JOB_COMPLETED", "data": {"device": "job0", "len":
> > > 134217728, "offset": 134217728, "speed": 0, "type": "commit"}}
> > > Â {"timestamp": {"seconds":Â TIMESTAMP, "microseconds":Â TIMESTAMP},
> > > "event": "JOB_STATUS_CHANGE", "data": {"status": "concluded", "id":
> > > "job0"}}
> > > Â {"timestamp": {"seconds":Â TIMESTAMP, "microseconds":Â TIMESTAMP},
> > > "event": "JOB_STATUS_CHANGE", "data": {"status": "null", "id":
> > > "job0"}}
> > > +{"return": {"status": "running", "singlestep": false, "running": true}}
> > > Â {"return": {}}
> > > Â {"timestamp": {"seconds":Â TIMESTAMP, "microseconds":Â TIMESTAMP},
> > > "event": "SHUTDOWN", "data": {"guest": false, "reason":
> > > "host-qmp-quit"}}
> > > Â *** done
> > >
> >
> >
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK