Re: [libvirt] [PATCH 12/20] Attach encryption information to virStorageVolDef.

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

 



----- "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
> >    <define name='vol'>
> >      <element name='volume'>
> > @@ -74,6 +76,7 @@
> >        </optional>
> >        <ref name='format'/>
> >        <ref name='permissions'/>
> > +      <ref name='encryption'/>
> >      </element>
> >    </define>
> 
> To allow removal of "<encryption>" for non-encrypted cases, I
> believe we'd need to add   
> 
>   <optional>
>     <ref name='encrption'/>
>   </optional>
> 
> Unless that's 'encryption' schema rule itself has <optional>
> already ?
Yes, it currently does.  I'll move the <optional> to the users, to be more explicit.

Thanks for the review,
   Mirek

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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]