After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 690779 - Boxes shouldn't let the user eject the installer ISO while an (unattended) installation is going on
Boxes shouldn't let the user eject the installer ISO while an (unattended) in...
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: installer
3.7.x
Other Linux
: Normal normal
: --
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks: 686781
 
 
Reported: 2012-12-27 16:31 UTC by Christophe Fergeau
Modified: 2016-03-31 13:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
libvirt-machine-props: Separate method for cdrom prop adding (9.72 KB, patch)
2013-03-12 01:22 UTC, Zeeshan Ali
committed Details | Review
Don't provide option to eject CDROM during installation (2.47 KB, patch)
2013-03-12 01:23 UTC, Zeeshan Ali
committed Details | Review
property: API to set sensitivity of prop widgets (919 bytes, patch)
2013-03-12 01:23 UTC, Zeeshan Ali
committed Details | Review
Disallow changing of system props during install (2.30 KB, patch)
2013-03-12 01:23 UTC, Zeeshan Ali
committed Details | Review

Description Christophe Fergeau 2012-12-27 16:31:32 UTC
Start an unattended installation, go to the VM properties and then to the 'devices' section. There's a 'remove' button next to the installation ISO image. I've checked that clicking on it after starting an unattended win7 install causes the installation to fail.
Comment 1 Zeeshan Ali 2013-03-12 01:22:57 UTC
Created attachment 238649 [details] [review]
libvirt-machine-props: Separate method for cdrom prop adding

Refactor code adding CDROM property into a separate function.
Comment 2 Zeeshan Ali 2013-03-12 01:23:13 UTC
Created attachment 238650 [details] [review]
Don't provide option to eject CDROM during installation

Both installation and live sessions require the installer/live media to
be present and therefore it does not make sense to allow users to eject
it during installation/live session. User could accidently click the
'Remove' button and everythig goes boom.
Comment 3 Zeeshan Ali 2013-03-12 01:23:17 UTC
Created attachment 238651 [details] [review]
property: API to set sensitivity of prop widgets
Comment 4 Zeeshan Ali 2013-03-12 01:23:20 UTC
Created attachment 238652 [details] [review]
Disallow changing of system props during install

Changing of system properties require a reboot and rebooting the machine
in the middle of an installation/live sesison is not a good idea. More so
since we track end of installation/live session through number of
reboots, epecially in case of live session where we automatically remove
the machine on shutdown/reboot if no installation was performed.
Comment 5 Christophe Fergeau 2013-03-13 09:48:50 UTC
Review of attachment 238652 [details] [review]:

s/sesison/session/
s/epecially/especially/
Comment 6 Zeeshan Ali 2013-03-13 13:46:33 UTC
Attachment 238649 [details] pushed as 46c4835 - libvirt-machine-props: Separate method for cdrom prop adding
Attachment 238650 [details] pushed as 4bed1f5 - Don't provide option to eject CDROM during installation
Attachment 238651 [details] pushed as 6af8b9b - property: API to set sensitivity of prop widgets
Attachment 238652 [details] pushed as 59dbcc5 - Disallow changing of system props during install