[libvirt] [PATCH] Add TCK script for verifying disk based storage pools

Eric Blake eblake at redhat.com
Wed Mar 3 18:54:28 UTC 2010


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"?

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 320 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100303/1c7b2e5b/attachment-0001.sig>


More information about the libvir-list mailing list