[et-mgmt-tools] [PATCH] report block size in add hardware summary

Adam Stokes astokes at redhat.com
Mon Aug 4 10:47:42 UTC 2008


> First this will need to be duplicated in create.py for the 
> guest creation wizard. I'd recommend breaking it out into
> it's own function as well so we don't start adding a lot
> of logic to the relatively simple get_config_* functions.
This is complete, though for the logic part im not really sure the best
way to handle the different varieties that can be seen here
> 
> Also, could you wrap this code in a try...except block, just
> log an error if we can't detect the size, and return None.
Complete as well 

> Actually could you just bring this up in bz 453061? It's basically
> covering the same issue.

Sure thing I've appended this issue to the above BZ.

Please let me know your thoughts on the latest patch

Thanks
-- 
              _|-  |  _  _
             _\|_()|<(/__\
 .----------------------------------.
( astokes at redhat.com || 919.754.4187 )
 `----------------------------------'
-------------- next part --------------
A non-text attachment was scrubbed...
Name: virt-manager-block-summary.patch
Type: text/x-patch
Size: 2257 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/et-mgmt-tools/attachments/20080804/1efda8c2/attachment.bin>


More information about the et-mgmt-tools mailing list