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

Miloslav Trmac mitr at redhat.com
Fri Aug 7 14:23:04 UTC 2009


----- "Daniel P. Berrange" <berrange at redhat.com> 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




More information about the libvir-list mailing list