[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Fix iotest 153
From: |
Maxim Levitsky |
Subject: |
Re: [PATCH] Fix iotest 153 |
Date: |
Mon, 04 May 2020 16:11:29 +0300 |
On Mon, 2020-05-04 at 15:08 +0200, Max Reitz wrote:
> On 04.05.20 11:41, Maxim Levitsky wrote:
> > On Mon, 2020-05-04 at 11:22 +0200, Max Reitz wrote:
> > > On 03.05.20 18:49, Maxim Levitsky wrote:
> > > > Commit f62514b3def5fb2acbef64d0e053c0c31fa45aff made qemu-img reject -o
> > > > "" but this test uses it
> > > >
> > > > Since this test only tries to do a dry-run run of qemu-img amend,
> > > > replace the -o "" with
> > > > dummy -o "size=0" since due to the nature of the test, it is not going
> > > > to reach the actual amend operation anyway
> > > >
> > > > Fixes: f62514b3def5fb2acbef64d0e053c0c31fa45aff
> > > >
> > > > Signed-off-by: Maxim Levitsky <address@hidden>
> > > > ---
> > > > tests/qemu-iotests/153 | 2 +-
> > > > tests/qemu-iotests/153.out | 12 ++++++------
> > > > 2 files changed, 7 insertions(+), 7 deletions(-)
> > > >
> > > > diff --git a/tests/qemu-iotests/153 b/tests/qemu-iotests/153
> > > > index 2b13111768..3f5029dd8f 100755
> > > > --- a/tests/qemu-iotests/153
> > > > +++ b/tests/qemu-iotests/153
> > > > @@ -122,7 +122,7 @@ for opts1 in "" "read-only=on"
> > > > "read-only=on,force-share=on"; do
> > > > _run_cmd $QEMU_IMG check $L "${TEST_IMG}"
> > > > _run_cmd $QEMU_IMG compare $L "${TEST_IMG}" "${TEST_IMG}"
> > > > _run_cmd $QEMU_IMG map $L "${TEST_IMG}"
> > > > - _run_cmd $QEMU_IMG amend -o "" $L "${TEST_IMG}"
> > > > + _run_cmd $QEMU_IMG amend -o "size=0" $L "${TEST_IMG}"
> > >
> > > AFAIU we don’t want this command to actually change the image (hence the
> > > empty options list, which would result in nothing being changed), so
> > > maybe "size=$size" would be more in the spirit of the test?
> >
> > This is a good idea! Should I resend the patch or you can add this change
> > locally?
>
> Fixing it up locally should actually rather be the exception than the
> rule. It makes sense for larger series where resending it just for one
> little thing generates too much noise, but in this case I don’t think
> sending a v2 would hurt.
>
> Max
>
V2 is on the way then!
Best regards,
Maxim Levitsky