Re: PATCH: 0/16: Storage management APIs

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

 



On Thu, Feb 14, 2008 at 05:51:13AM -0500, Daniel Veillard wrote:
> On Tue, Feb 12, 2008 at 04:28:29AM +0000, Daniel P. Berrange wrote:
> > The patch series that follows provides storage management APIs for dealing
> > with local directories, local filesystems, remote filesystems, logical
> > volumes (LVM), disk partitioning, and iSCSI. It will also soon support
> > SCSI host adapters (including NPIV).
> > 
> > I'll include comments inline with each patch. The final patch in the
> > series also provides a 'storage.html' page for the website giving a fairly
> > friendly overview of the storage pool types and their associated XML
> > examples.
> > 
> > Although included here, the one API i really don't like is the one for
> > discovery virConnectDiscoverStoragePools, since its API is not flexible
> > enough in terms of metadata it accepts. I will probably just leave this
> > out next time around, since its not critical for the core functionality
> > and can thus be  added later.
> > 
> > Also TBD is a way to format a filesystem on a  volume, and how to clone
> > an existing volume, and take a snapshot. These things can all be added
> > at a later date.
> > 
> > Finally it does not include the async job support. This is also best added
> > once the core code is merged.
> 
>   In general I like the approach 'let's put what we consider fine first
> and discuss other parts later' . I guess it's the third round of review
> and I really find easier to comment based on existing stuff than big patches
> that gets iterated over time. So let's isolate what we consider okay
> and not needing further discussion, push them now, than in an upcoming
> fourth iteration, 

With the exception of the virConnectDiscoverSToragePools api I'm happy
for any of this to be committed - so people just ACK what you want to
see in CVS & I'll make it so...

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

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