Re: [PATCH] report block size in add hardware summary

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Adam Stokes wrote:
> The way virt-manager is setup now is that it only reports sizes of file
> backed images and a "-" with storage partitions.

I'm fine with the idea, however this patch needs a bit of work.

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.

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.

> 
> This patch addresses that issue.
> 
> I'll be filing a bug for RHEL5 as soon as bugzilla comes out of
> maintenance mode
> 

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

Thanks,
Cole



_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux