GNOME Bugzilla – Bug 677101
VM view state lost on VM reboot
Last modified: 2016-03-31 13:57:47 UTC
- start a Windows XP unattended install (probably works with other OSes) - click on the VM icon to get its display maximized - wait until Windows installer reboots Result: - the VM display is back to iconized Expected result: - VM is still maximized
During vm-creator time, the machine is shutdown, and restarted with post_install_setup (). A shutdown will lead to a display disconnect, which will trigger a UI switch back to collection view. We need a special case when the machine is first shutdown during unattended installation. It also need to handle the case where the shutdown is caused by installation failure.
assigning to me, as I started a bit on that
(In reply to comment #1) > During vm-creator time, the machine is shutdown, and restarted with > post_install_setup (). A shutdown will lead to a display disconnect, which will > trigger a UI switch back to collection view. We need a special case when the > machine is first shutdown during unattended installation. It also need to > handle the case where the shutdown is caused by installation failure. Right. Keep some things in mind though: 1. Solution doesnt need to be limited to unattended installs. 2. We have custom XML in domain config that tells if installation is in progress. 3. Wonder how to deal with OSs that reboot multiple times during installations, e.g windows.
(In reply to comment #1) > During vm-creator time, the machine is shutdown, and restarted with > post_install_setup () Why is that? Shouldn't/couldn't it be reset instead?
(In reply to comment #4) > (In reply to comment #1) > > During vm-creator time, the machine is shutdown, and restarted with > > post_install_setup () > > Why is that? Shouldn't/couldn't it be reset instead? We need to change the configuration after first boot so we ask libvirt to shutdown on reset for first boot.
Created attachment 216094 [details] [review] Stay on Display during wizard reboot During unattended installation, the VM may be rebooted.
note: the patch is wip and not tested thoroughly.
I didn't review this patch because you said in the latest commit that patch is WIP and not tested thoroughly so I was waiting on you to provide an improved version or inform me if its good in your opinion.
(In reply to comment #8) > I didn't review this patch because you said in the latest commit that patch is > WIP and not tested thoroughly so I was waiting on you to provide an improved > version or inform me if its good in your opinion. That patch is part of https://bugzilla.gnome.org/show_bug.cgi?id=680293, and has been reviewed.
I think this is mostly fixed by now.