Bug 748068

Summary: virt-manager should warn about performance implications of not specifying a guest OS
Product: [Fedora] Fedora Reporter: Dave Allan <dallan>
Component: virt-managerAssignee: Cole Robinson <crobinso>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: rawhideCC: berrange, crobinso, dougsland, dpierce, hbrock, jforbes, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-14 00:02:01 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dave Allan 2011-10-21 21:58:08 UTC 
Description of problem:
virt-manager defaults to generic OS, preventing the use of optimized i/o and often resulting in greatly decreased guest performance.  The create VM flow should be changed to require the user to choose an OS variant and then should warn about the performance implications if the user chooses generic.

Version-Release number of selected component (if applicable):
0.9.0 (actually upstream git repo and F14 tested)
Comment 1 Cole Robinson 2012-02-13 23:40:34 UTC 
I'd like to get there eventually, but right now since our distro detection is pretty weak, it's a pain to force this on users (and this was originally signed off by UI designers).

Once we start using libosinfo and have much better OS detection and metadata, I think making this required is more feasible. And we are probably nearing the point where we can assume virtio it is available for anything being installed from URL or PXE.

For now I put a warning message at the end of the install wizard if the user didn't specify an OS:

http://git.fedorahosted.org/git?p=virt-manager.git;a=commit;h=1e10c830b9608dcf36af4e40298049c46f21f09c
Comment 2 Cole Robinson 2012-02-14 00:02:01 UTC 
Actually since the UI files have changed upstream, pulling this into rawhide would require a custom backport which just isn't worth it since this will prob be in fedora with a rebase in a month. So just closing as UPSTREAM