On 03/20/2013 11:33 PM, Chris Evich wrote:
On 03/20/2013 06:11 AM, Yu Mingfei wrote:
virsh_instance = virsh.VirshPersistent(**virsh_dargs)

IIRC, you can recycle this by calling virsh_instance.new_session() here, but it's only very small nit-pick, what you have is fine. The test looks good overall.

Yes, good idea. I will replace it before committed.


ToDo: IIRC, there's a way to reboot the host and have autotest resume testing when it comes up. I'm almost 100% sure this doesn't work with current virt-test code, but it would be nice to get it working. This would be an ideal test to play with the host-reboot functionality. Since this is the central-reason why autostart exists, and at some point we will need to test the guest-autostart feature :)

In fact, My first thought about net-autostart is:
1.With setting autostart, the network will be active after I reboot libvirtd
2.With setting disable, the network will be inactive after libvirtd is restarted.
   (Even though the network is active.)

Then I found if network is active, no matter I set autostart or disable,
the network will still be running after I restart libvirtd. :D
So rebooting host may be a more suitable choice.

But currently I just destroy the default network before I run testcases,
then check it after restarting libvirtd.

And about checking libvirt network's state by reboot host,
I think we can replace it by restarting network service.

Thanks~

--
Best Regards
Yu Mingfei

_______________________________________________
Virt-test-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-test-devel

Reply via email to