Re: [libvirt PATCH] examples: fix storage volume size schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 06, 2020 at 12:47:23 +0200, Peter Krempa wrote:
> On Tue, Oct 06, 2020 at 10:22:27 +0100, Daniel Berrange wrote:
> > There is no such <storage> element, <capacity> and <allocation> exist at
> > the top level.
> > 
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> > ---
> >  examples/xml/storage/vol-cow.xml    | 6 ++----
> >  examples/xml/storage/vol-qcow.xml   | 6 ++----
> >  examples/xml/storage/vol-qcow2.xml  | 6 ++----
> >  examples/xml/storage/vol-raw.xml    | 6 ++----
> >  examples/xml/storage/vol-sparse.xml | 6 ++----
> >  examples/xml/storage/vol-vmdk.xml   | 6 ++----
> >  6 files changed, 12 insertions(+), 24 deletions(-)
> 
> Definitely an improvement but some of them still don't validate:
> 
> jing ../../../docs/schemas/storagevol.rng vol*
> /home/pipo/libvirt/examples/xml/storage/vol-raw.xml:5:10: error: element "volume" incomplete; missing required element "target"
> /home/pipo/libvirt/examples/xml/storage/vol-sparse.xml:5:10: error: element "volume" incomplete; missing required element "target"
> 
> 
> I'll probably fix those along with addition to virschematest to prevent
> this from happening.

Forgot to add:

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux