-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7307/#review11963
-----------------------------------------------------------

Ship it!


Ship It!

- edison su


On Sept. 27, 2012, 4:23 a.m., Marcus Sorensen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7307/
> -----------------------------------------------------------
> 
> (Updated Sept. 27, 2012, 4:23 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Description
> -------
> 
> This should alleviate reports that the host OS has to be rebooted after agent 
> install because host says that HVM isn't supported. Libvirtd doesn't seem to 
> need to be restarted, it reports capabilities accordingly.
> 
> 
> Diffs
> -----
> 
>   cloud.spec 67064b3 
> 
> Diff: https://reviews.apache.org/r/7307/diff/
> 
> 
> Testing
> -------
> 
> Verified that kvm.modules is run after agent install if it exists and is 
> executable. I don't think libvirtd needs to be restarted since I also tested 
> what a capabilities call shows pre and post modprobe, and it changes 
> accordingly without a libvirtd restart.
> 
> 
> Thanks,
> 
> Marcus Sorensen
> 
>

Reply via email to