Re: [PATCH v2 1/2] iotests: add another bash sleep command to 247

[Andrey Shinkevich]

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.

> > 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