[Ovirt-devel] [PATCH] Host Status Patches

[Scott Seago]

Ian Main wrote:
> A few comments first:
>
> There will likely be some fallout from this for the UI and maybe some vm task modifications.  This has had some light testing but may still have issues.  I just want to get it out there so we can work together to wrap this up.  
>
> This was developed and tested on 0.4 and then moved to tip but was having problems getting it running.  I'll test it more on tip in the morning.
>
> ---
>
> This patch gets host-status working properly and adds a new STATE_UNREACHABLE to the VM states.  This state is set when a managed host with VMs is no longer reachable and we're not sure if it died or fell off the network.  Once the host is back up the actual state of the VMs can be inferred and taskomatic will be informed.
>
> Signed-off-by: Ian Main <imain at stemwinder.org>
>   
I guess we'll need to work out how to handle this on the UI. If we're 
planning on letting users "stop" unreachable VMs, then we'll want to add 
this to the VALID_ACTIONS_PER_VM_STATE hash in vm_task.rb -- if  this 
should be an admin-only function, then we will need to work out the 
permission/role rules for this and where it will go in the UI.

Other than that, this looks OK from the model side of things.

Scott