Created attachment 854262 [details]
Engine log

Description of problem:
OSInfo file luck validation for legal values

For example changing the following 
os.windows_8.devices.display.protocols.value = vnc/cirrus
to
os.windows_8.devices.display.protocols.value = aaa/bbb

Throws the following exception

2014-01-23 10:52:13,079 ERROR [org.ovirt.engine.core.bll.OsRepositoryQuery] (ajp--127.0.0.1-8702-2) Query OsRepositoryQuery failed. Exception message is No enum constant org.ovirt.engine.core.common.businessentities.DisplayType.aaa : java.lang.IllegalArgumentException: No enum constant org.ovirt.engine.core.common.businessentities.DisplayType.aaa: java.lang.IllegalArgumentException: No enum constant org.ovirt.engine.core.common.businessentities.DisplayType.aaa


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.Open the osinfo file 
2.change os.windows_8.devices.display.protocols.value = vnc/cirrus
to
os.windows_8.devices.display.protocols.value = aaa/bbb
3.restart ovirt-engine service 
4.try to create a new VM


Actual results:
An exception is thrown , most fields in the New VM dialog displayed with blank data 

Expected results:
Basic validation should be performed in order to handle illegal data properly

Additional info:

This is general for the whole osinfo file settings , the display value that was changed is just an example but this will occur with other entries as well