[libvirt] [PATCH] Add TCK script for verifying disk based storage pools
Daniel P. Berrange
berrange at redhat.com
Thu Mar 4 10:26:16 UTC 2010
On Wed, Mar 03, 2010 at 11:54:28AM -0700, Eric Blake wrote:
> According to Daniel P. Berrange on 3/3/2010 11:46 AM:
>
> High-level review nits below. However, I still count myself as too
> inexperienced to offer either an ACK or NAK on this.
>
> > This adds a reasonably comprehensive test script for working with
> > disk based storage pools. It checks
> >
> > - Prevent re-formating an existing partition table (currently
>
> s/formating/formatting/ (multiple instances)
>
> > missing this check in libvirt)
> > - Formating of a partiton table
> > - Wiping a partition table (currently missing in libvirt)
> > - Creating multiple storage volumes
> > - Deleteing storage volumes
>
> s/Deleteing/Deleting/
>
> > +# List of host block devices that the test suite can trash all
> > +# data on, for purposes of testing. You have been warned !
>
> Indeed!
>
> > @@ -0,0 +1,127 @@
> > +# -*- perl -*-
> > +#
> > +# Copyright (C) 2009-2010 Red Hat
> > +# Copyright (C) 2009-2010 Daniel P. Berrange
>
> Shouldn't the canonical copyright be "Red Hat, Inc.", and not just "Red Hat"?
Yep, you're right
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
More information about the libvir-list
mailing list