[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/8] nbd: Add some error case testing to iote
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/8] nbd: Add some error case testing to iotests 223 |
Date: |
Mon, 14 Jan 2019 08:25:57 +0000 |
11.01.2019 22:47, Eric Blake wrote:
> Testing success paths is important, but it's also nice to highlight
> expected failure handling, to show that we don't crash, and so that
> upcoming tests that change behavior can demonstrate the resulting
> effects on error paths.
>
> Add the following errors:
> Attempting to export without a running server
> Attempting to start a second server
> Attempting to export a bad node name
> Attempting to export a name that is already exported
> Attempting to export an enabled bitmap
> Attempting to clean an already cleaned export
For me "clean" sounds not the same as "remove" (I used to the thought that
remove means to remove object from larger object, and clean is clean the
object internally (remove its content?), but it continues to exist and all
its relations with parents are not touched), but I'm really far from being
an expert in English.
> Attempting to quit server a second time
>
> All of these properly complain except for a second server-stop,
> which will be fixed next.
hm, looks a bit strange to add test first and then fix bug, but on the
other hand it's more demonstrative, OK for me.
>
> Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
--
Best regards,
Vladimir
- [Qemu-devel] [PATCH v3 0/8] Promote x-nbd-server-add-bitmap to stable, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 2/8] nbd: Forbid nbd-server-stop when server is not running, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 3/8] nbd: Only require disabled bitmap for read-only exports, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 4/8] nbd: Merge nbd_export_set_name into nbd_export_new, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 7/8] nbd: Merge nbd_export_bitmap into nbd_export_new, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 5/8] nbd: Allow bitmap export during QMP nbd-server-add, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 1/8] nbd: Add some error case testing to iotests 223, Eric Blake, 2019/01/11
- Re: [Qemu-devel] [PATCH v3 1/8] nbd: Add some error case testing to iotests 223,
Vladimir Sementsov-Ogievskiy <=
- [Qemu-devel] [PATCH v3 8/8] qemu-nbd: Add --bitmap=NAME option, Eric Blake, 2019/01/11
- [Qemu-devel] [PATCH v3 6/8] nbd: Remove x-nbd-server-add-bitmap, Eric Blake, 2019/01/11
- Re: [Qemu-devel] [PATCH v3 0/8] Promote x-nbd-server-add-bitmap to stable, Eric Blake, 2019/01/11