[Libvir] PATCH: 0/16: Storage management APIs
Daniel P. Berrange
berrange at redhat.com
Thu Feb 14 15:22:31 UTC 2008
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 -=|
More information about the libvir-list
mailing list