[Ovirt-devel] Refactoring hosts table for updated CPU info

[Scott Seago]

Hugh O. Brock wrote:
> On Mon, Jun 30, 2008 at 10:21:28AM -0400, Darryl L. Pierce wrote:
>   
>> In reviewing the CPU details that are being reported from the managed  
>> node, I'd like to propose the following changes to accomodate the  
>> enhanced details.
>>
>> 1. Since machines will have more than one CPU, and each CPU can have  
>> different values that are to be reported, the CPU details for the host  
>> will be moved to a separate table (cpu_details) and loaded with the host  
>> as needed.
>>
>> 2. Currently the host screen shows the number of CPUs and the speed as a  
>> single value. We could drop the the speed column and, if the user clicks  
>> on the CPU count cell, drop down a set of rows that enumerate the CPUs  
>> and their details.
>>
>> 3. The displayed list will give the enhanced set of CPU details being  
>> reported during the identify phase.
>>
>> Thoughts?
>>
>>     
>
> This seems reasonable to me but I will defer to Scott, who is the
> Keeper of the Model more or less...
>
> Scott?
>   
Heh -- I was just about to reply.

Are there parameters that can be different on CPUs for a given hosts and 
others that must be identical? If some will never difer, those might 
need to stick with the Host record.
Otherwise I"m fine with this. We'd probably want to preserve the ability 
to get num_cpus from the host -- this would just return cpus.size.

I don't think we'd want to mess with clickable count cells in the 
flexigrid UI, as it might confuse the existing click action -- which 
displays the detail pane. We _would_ want to include full CPU details in 
the detail pane though -- so when the user clicked on the host row (CPU 
field or otherwise) the details pane would show full CPU info.

Do you have the full list of attributes you're going to add per CPU?

Also, there has been talk of refactoring the host NIC model as well -- 
are you working on this along with the CPU bits, or is that an unrelated 
stream of work?

Scott