Re: [PATCH] storage: zfs: implement pool build and delete

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

 



  Roman Bogorodskiy wrote:

>   Ján Tomko wrote:
> 
> > On 09/07/2014 04:15 PM, Roman Bogorodskiy wrote:
> > > Provide an implementation for buildPool and deletePool operations for
> > > the ZFS storage backend.
> > > ---
> > >  docs/schemas/storagepool.rng      |  1 +
> > >  src/storage/storage_backend_zfs.c | 57 +++++++++++++++++++++++++++++++++++++++
> > >  2 files changed, 58 insertions(+)
> > > 
> > 
> > ACK.
> 
> Thanks; will push closer to the weekend.
> 
> > > diff --git a/docs/schemas/storagepool.rng b/docs/schemas/storagepool.rng
> > > index 908cc11..da8e1f3 100644
> > > --- a/docs/schemas/storagepool.rng
> > > +++ b/docs/schemas/storagepool.rng
> > > @@ -386,6 +386,7 @@
> > >      <element name='source'>
> > >        <interleave>
> > >          <ref name='sourceinfoname'/>
> > > +        <ref name='sourceinfodev'/>
> > >        </interleave>
> > >      </element>
> > >    </define>
> > 
> > It would be nice to add an example to docs/formatstorage.html or storage.html.
> > Also an XML to the test suite to check its validity against the RNG schema.
> 
> Yes, that's true; added to my todo list.
> 
> Also, I think it'd be good to add a domain xml example to the bhyve
> driver page and ideally create some unit tests for the ZFS backend code.

I decided to add a test before pushing to make sure that everything is
right and... here's a result, I've found some minor problems. So I've
rolled a v2 with the tests included and issues fixed:

http://www.redhat.com/archives/libvir-list/2014-September/msg00829.html

Should not be lazy to add tests first as they do the job.

Roman Bogorodskiy

Attachment: pgpujzt_gmHwI.pgp
Description: PGP signature

--
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]
  Powered by Linux