On Fri, Feb 14, 2020 at 01:50:22PM +0100, Peter Krempa wrote:
On Thu, Feb 13, 2020 at 15:54:53 +0100, Ján Tomko wrote:
> On Wed, Feb 12, 2020 at 07:03:23PM +0100, Peter Krempa wrote:
> > Add a set of test data to see whether the backing store strings are
> > formatted reasonably. Note that we don't support direct creation of such
> > images so those tests are not enabled.
> >
> > Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
> > ---
> > tests/qemublocktest.c | 2 ++
> > .../imagecreate/qcow2-backing-qcow2-slice.json | 15 +++++++++++++++
> > .../imagecreate/qcow2-backing-qcow2-slice.xml | 1 +
> > .../imagecreate/qcow2-backing-raw-slice.json | 15 +++++++++++++++
> > .../imagecreate/qcow2-backing-raw-slice.xml | 1 +
> > .../qemublocktestdata/imagecreate/qcow2-slice.xml | 14 ++++++++++++++
> > tests/qemublocktestdata/imagecreate/raw-slice.xml | 14 ++++++++++++++
> > 7 files changed, 62 insertions(+)
> > create mode 100644
tests/qemublocktestdata/imagecreate/qcow2-backing-qcow2-slice.json
> > create mode 120000
tests/qemublocktestdata/imagecreate/qcow2-backing-qcow2-slice.xml
> > create mode 100644
tests/qemublocktestdata/imagecreate/qcow2-backing-raw-slice.json
> > create mode 120000
tests/qemublocktestdata/imagecreate/qcow2-backing-raw-slice.xml
> > create mode 100644 tests/qemublocktestdata/imagecreate/qcow2-slice.xml
> > create mode 100644 tests/qemublocktestdata/imagecreate/raw-slice.xml
> >
> > diff --git a/tests/qemublocktestdata/imagecreate/qcow2-slice.xml
b/tests/qemublocktestdata/imagecreate/qcow2-slice.xml
> > new file mode 100644
> > index 0000000000..6c5ae3107b
> > --- /dev/null
> > +++ b/tests/qemublocktestdata/imagecreate/qcow2-slice.xml
> > @@ -0,0 +1,14 @@
> > +<disk device='disk' name='vda'>
> > + <driver type='qcow2'/>
> > + <source file='/var/lib/libvirt/images/i.qcow2'>
> > + <slices>
> > + <slice type='storage' offset='1234'
size='5768'/>
> > + </slices>
> > + <privateData>
> > + <nodenames>
> > + <nodename type='storage'
name='0123456789ABCDEF0123456789ABCDE'/>
>
> > + <nodename type='format'
name='0123456789ABCDEF0123456789ABCDE'/>
>
> This is not parsed.
>
> > + </nodenames>
> > + </privateData>
> > + </source>
> > +</disk>
> > diff --git a/tests/qemublocktestdata/imagecreate/raw-slice.xml
b/tests/qemublocktestdata/imagecreate/raw-slice.xml
> > new file mode 100644
> > index 0000000000..adc7a175ce
> > --- /dev/null
> > +++ b/tests/qemublocktestdata/imagecreate/raw-slice.xml
> > @@ -0,0 +1,14 @@
> > +<disk device='disk' name='vda'>
> > + <driver type='raw'/>
> > + <source file='/var/lib/libvirt/images/i.img'>
> > + <slices>
> > + <slice type='storage' offset='9876'
size='54321'/>
> > + </slices>
> > + <privateData>
> > + <nodenames>
> > + <nodename type='storage'
name='0123456789ABCDEF0123456789ABCDE'/>
>
> > + <nodename type='format'
name='0123456789ABCDEF0123456789ABCDE'/>
>
> This is not parsed either.
Both are parsed. These are nodenames of the storage and format layer
when instantiating storage via -blockdev.
I sit corrected.
Re-reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano
>
>
> > + </nodenames>