[et-mgmt-tools] [patch] virt-image verify disk checksum

Cole Robinson crobinso at redhat.com
Mon Oct 20 14:32:06 UTC 2008


Joey Boggs wrote:
> This patch will make use of the recent disk checksum function for 
> virt-image and verifies prior to importing
> 

> diff -r 9f45a36d8242 virt-image
> --- a/virt-image	Thu Oct 16 11:18:49 2008 -0400
> +++ b/virt-image	Fri Oct 17 16:12:16 2008 -0400
> @@ -145,6 +145,8 @@
>      parser.add_option("", "--replace",action="store_true", dest="replace",
>                        help=_("Overwrite, or destroy, an existing image with the same name"),
>                        default=False)                      
> +    parser.add_option("","--skip-checksum", action="store_true", dest="skipchecksum",
> +                      help=_("Skip disk checksum verification process"))
>  
>      (options,args) = parser.parse_args()
>      if len(args) < 1:
> @@ -204,6 +206,10 @@
>          if options.noapic:
>              guest.features["apic"] = False
>  
> +    if not options.skipchecksum:
> +        for d in image.storage.values():
> +            virtinst.ImageParser.Disk.check_disk_signature(d)
> +    

This line just abuses the way python does object orientation. You
want to do:

    if not options.skipchecksum:
        for disk in image.storage.values():
            disk.check_disk_signature()

Thanks,
Cole





More information about the et-mgmt-tools mailing list