* [Qemu-devel] [PATCH] qemu-iotests: Remove _supported_fmt dmg
@ 2018-04-10 8:42 Kevin Wolf
2018-04-10 12:36 ` Eric Blake
0 siblings, 1 reply; 2+ messages in thread
From: Kevin Wolf @ 2018-04-10 8:42 UTC (permalink / raw)
To: qemu-block; +Cc: kwolf, qemu-devel
qemu-iotests doesn't support dmg, and the dmg block driver doesn't
support image creation. Two test cases declare dmg as supported, but
that's obviously wrong for both reasons. Remove the declaration.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
tests/qemu-iotests/183 | 2 +-
tests/qemu-iotests/194 | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/qemu-iotests/183 b/tests/qemu-iotests/183
index 20268ff7a1..c49e1ad6ef 100755
--- a/tests/qemu-iotests/183
+++ b/tests/qemu-iotests/183
@@ -43,7 +43,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
. ./common.filter
. ./common.qemu
-_supported_fmt qcow2 raw qed dmg quorum
+_supported_fmt qcow2 raw qed quorum
_supported_proto file
_supported_os Linux
diff --git a/tests/qemu-iotests/194 b/tests/qemu-iotests/194
index 1d4214aca3..d746ab1e21 100755
--- a/tests/qemu-iotests/194
+++ b/tests/qemu-iotests/194
@@ -21,7 +21,7 @@
import iotests
-iotests.verify_image_format(supported_fmts=['qcow2', 'qed', 'raw', 'dmg'])
+iotests.verify_image_format(supported_fmts=['qcow2', 'qed', 'raw'])
iotests.verify_platform(['linux'])
with iotests.FilePath('source.img') as source_img_path, \
--
2.13.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [PATCH] qemu-iotests: Remove _supported_fmt dmg
2018-04-10 8:42 [Qemu-devel] [PATCH] qemu-iotests: Remove _supported_fmt dmg Kevin Wolf
@ 2018-04-10 12:36 ` Eric Blake
0 siblings, 0 replies; 2+ messages in thread
From: Eric Blake @ 2018-04-10 12:36 UTC (permalink / raw)
To: Kevin Wolf, qemu-block; +Cc: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 612 bytes --]
On 04/10/2018 03:42 AM, Kevin Wolf wrote:
> qemu-iotests doesn't support dmg, and the dmg block driver doesn't
> support image creation. Two test cases declare dmg as supported, but
> that's obviously wrong for both reasons. Remove the declaration.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> tests/qemu-iotests/183 | 2 +-
> tests/qemu-iotests/194 | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 619 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-04-10 12:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-04-10 8:42 [Qemu-devel] [PATCH] qemu-iotests: Remove _supported_fmt dmg Kevin Wolf
2018-04-10 12:36 ` Eric Blake
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).