From: Peter Lieven <pl@kamp.de>
To: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>, Fam Zheng <famz@redhat.com>,
ronniesahlberg@gmail.com, Jeff Cody <jcody@redhat.com>,
Peter Lieven <pl@kamp.de>, Max Reitz <mreitz@redhat.com>,
owasserm@redhat.com, Federico Simoncelli <fsimonce@redhat.com>,
Stefan Hajnoczi <stefanha@redhat.com>,
Wenchao Xia <xiawenc@linux.vnet.ibm.com>
Subject: [Qemu-devel] [PATCHv2 00/18] qemu-iotests: adjust tests to work with the NFS protocol
Date: Sun, 5 Jan 2014 18:21:50 +0100 [thread overview]
Message-ID: <1388942528-10453-1-git-send-email-pl@kamp.de> (raw)
In order to proceed with the integration of the NFS protocol driver into
qemu I was asked by Stefan to add integration for NFS into the qemu-iotests.
Unfortunately, this became more complex than I had expected because most
of the tests for non RAW formats only work with the file protocol because
they use shell commands like cp, rm or mv which obviously don't work
on nfs:// URLs.
This series first changes all those tests that are not working out of the
box to support only the file protocol.
After the NFS protocol is introduced in Patch 2 I fix most of them
to work with any protocol.
After this series the qemu-iotests for NFS run through gracefully with
the RAW, QCOW2 and VMDK formats.
There are 3 topics open:
- test 051 fails regardless which protocol is used (I already send a msg
to the list)
- test 052 should work, but it seems there is a bug in the bdrv_open
logic if the BDRV_O_SNAPSHOT flag is set and the protocol is anything
else than file. Maybe someone with more understanding of the whole
open logic could look at this. I do not believe that its sth which
has to do with the NFS driver since the test fails while opening the
backing file and other backing file tests run without problems.
- other protocols like sheepdog or ssh that are allowed to use other formats
than raw should be tested. they actually can't never have run qemu-iotests
with qcow2 protocol for instance.
If you want to do your tests please make sure to have
[PATCHv5] block: add native support for NFS
[PATCH v2] vmdk: Allow vmdk_create to work with protocol
Peter
Peter Lieven (18):
qemu-iotests: change _supported_proto to file for various tests
qemu-iotests: enable support for NFS protocol
qemu-iotests: enable test 016 and 025 to work with NFS protocol
qemu-iotests: fix test 013 to work with any protocol
qemu-iotests: fix tests 014 and 023 to work with any protocol
qemu-iotests: fix test 018 to work with any protocol
qemu-iotests: fix test 019 to work with any protocol
qemu-iotests: fix test 020 to work with any protocol
qemu-iotests: fix test 024 to work with any protocol
qemu-iotests: fix test 028 to work with any protocol
qemu-iotests: fix test 034 to work with any protocol
qemu-iotests: fix test 037 to work with any protocol
qemu-iotests: fix test 038 to work with any protocol
qemu-iotests: fix test 043 to work with any protocol
qemu-iotests: fix test 046 to work with any protocol
qemu-iotests: fix expected output of test 067
qemu-iotests: blacklist test 020 for NFS protocol
qemu-iotests: test 026 store blkdebug.cfg locally
tests/qemu-iotests/013 | 7 +++----
tests/qemu-iotests/013.out | 2 +-
tests/qemu-iotests/014 | 2 +-
tests/qemu-iotests/014.out | 2 +-
tests/qemu-iotests/016 | 2 +-
tests/qemu-iotests/018 | 20 ++++++++------------
tests/qemu-iotests/018.out | 4 ++--
tests/qemu-iotests/019 | 28 +++++++++++++---------------
tests/qemu-iotests/019.out | 4 ++--
tests/qemu-iotests/020 | 27 +++++++++++++++------------
tests/qemu-iotests/020.out | 2 +-
tests/qemu-iotests/023 | 9 ++++-----
tests/qemu-iotests/023.out | 16 ++++++++--------
tests/qemu-iotests/024 | 10 ++++------
tests/qemu-iotests/024.out | 4 ++--
tests/qemu-iotests/025 | 2 +-
tests/qemu-iotests/026 | 18 ++++++++++++------
tests/qemu-iotests/028 | 7 +++----
tests/qemu-iotests/028.out | 2 +-
tests/qemu-iotests/031 | 2 +-
tests/qemu-iotests/034 | 5 ++---
tests/qemu-iotests/034.out | 2 +-
tests/qemu-iotests/036 | 2 +-
tests/qemu-iotests/037 | 7 ++-----
tests/qemu-iotests/037.out | 2 +-
tests/qemu-iotests/038 | 7 +++----
tests/qemu-iotests/038.out | 2 +-
tests/qemu-iotests/039 | 2 +-
tests/qemu-iotests/043 | 22 +++++++++-------------
tests/qemu-iotests/043.out | 12 ++++++------
tests/qemu-iotests/046 | 7 ++-----
tests/qemu-iotests/046.out | 2 +-
tests/qemu-iotests/052 | 2 +-
tests/qemu-iotests/054 | 2 +-
tests/qemu-iotests/059 | 2 +-
tests/qemu-iotests/060 | 2 +-
tests/qemu-iotests/061 | 2 +-
tests/qemu-iotests/063 | 2 +-
tests/qemu-iotests/067.out | 8 ++++----
tests/qemu-iotests/069 | 2 +-
tests/qemu-iotests/common | 22 +++++++++++++++++++---
tests/qemu-iotests/common.pattern | 7 +++----
tests/qemu-iotests/common.rc | 3 +++
43 files changed, 150 insertions(+), 146 deletions(-)
--
1.7.9.5
next reply other threads:[~2014-01-05 17:22 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-05 17:21 Peter Lieven [this message]
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 01/18] qemu-iotests: change _supported_proto to file for various tests Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 02/18] qemu-iotests: enable support for NFS protocol Peter Lieven
2014-01-06 20:14 ` Jeff Cody
2014-01-06 22:19 ` Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 03/18] qemu-iotests: enable test 016 and 025 to work with " Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 04/18] qemu-iotests: fix test 013 to work with any protocol Peter Lieven
2014-01-06 5:31 ` Fam Zheng
2014-01-06 6:48 ` Peter Lieven
2014-01-06 10:09 ` Fam Zheng
2014-01-06 12:21 ` Peter Lieven
2014-01-06 12:47 ` Fam Zheng
2014-01-06 20:40 ` Jeff Cody
2014-01-06 22:35 ` Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 05/18] qemu-iotests: fix tests 014 and 023 " Peter Lieven
2014-01-06 5:40 ` Fam Zheng
2014-01-06 6:49 ` Peter Lieven
2014-01-10 19:04 ` Kevin Wolf
2014-01-10 19:06 ` Peter Lieven
2014-01-10 19:14 ` Kevin Wolf
2014-01-10 19:36 ` Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 06/18] qemu-iotests: fix test 018 " Peter Lieven
2014-01-06 5:45 ` Fam Zheng
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 07/18] qemu-iotests: fix test 019 " Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 08/18] qemu-iotests: fix test 020 " Peter Lieven
2014-01-05 17:21 ` [Qemu-devel] [PATCHv2 09/18] qemu-iotests: fix test 024 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 10/18] qemu-iotests: fix test 028 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 11/18] qemu-iotests: fix test 034 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 12/18] qemu-iotests: fix test 037 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 13/18] qemu-iotests: fix test 038 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 14/18] qemu-iotests: fix test 043 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 15/18] qemu-iotests: fix test 046 " Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 16/18] qemu-iotests: fix expected output of test 067 Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 17/18] qemu-iotests: blacklist test 020 for NFS protocol Peter Lieven
2014-01-06 5:51 ` Fam Zheng
2014-01-06 6:41 ` Peter Lieven
2014-01-05 17:22 ` [Qemu-devel] [PATCHv2 18/18] qemu-iotests: test 026 store blkdebug.cfg locally Peter Lieven
2014-02-24 12:54 ` [Qemu-devel] [PATCHv2 00/18] qemu-iotests: adjust tests to work with the NFS protocol Stefan Hajnoczi
2014-02-24 20:21 ` Peter Lieven
2014-02-25 10:49 ` Stefan Hajnoczi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1388942528-10453-1-git-send-email-pl@kamp.de \
--to=pl@kamp.de \
--cc=famz@redhat.com \
--cc=fsimonce@redhat.com \
--cc=jcody@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=owasserm@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=ronniesahlberg@gmail.com \
--cc=stefanha@redhat.com \
--cc=xiawenc@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).