Martin Peřina has posted comments on this change.

Change subject: core: WIP: Support for device custom properties
......................................................................


Patch Set 5: (3 inline comments)

....................................................
File backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
Line 523: -- Device Custom Properties
Line 524: select fn_db_add_config_value('SupportCustomDeviceProperties', 
'false', '3.0');
Line 525: select fn_db_add_config_value('SupportCustomDeviceProperties', 
'false', '3.1');
Line 526: select fn_db_add_config_value('SupportCustomDeviceProperties', 
'false', '3.2');
Line 527: select fn_db_add_config_value('CustomDeviceProperties', '', '3.3');
SupportCustomDeviceProperties is used to define, if these properties are 
supported on the oVirt version.
CustomDeviceProperties is used to store properties definition, which is used to 
validate properties value for the device (more information in 
http://www.ovirt.org/Features/Device_Custom_Properties#Configuration )
Line 528: 
Line 529: 
------------------------------------------------------------------------------------
Line 530: --                  Update with override section
Line 531: 
------------------------------------------------------------------------------------


....................................................
File backend/manager/modules/dal/src/test/resources/fixtures.xml
Line 4112:             <value>true</value>
Line 4113:             <value>true</value>
Line 4114:             <value>false</value>
Line 4115:             <value>alias</value>
Line 4116:             <value></value>
I will fix this, thanks.
Line 4117:         </row>
Line 4118:      </table>
Line 4119: 
Line 4120:     <table name="job">


....................................................
File 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
Line 659
Line 660
Line 661
Line 662
Line 663
It was caused by Eclipse, thanks for notice. I will fix it even more, because 
the whole if statement is completely unnecessary.


--
To view, visit http://gerrit.ovirt.org/14309
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I67ed453706ac75cdc4356cc2d60913d8958c89ed
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to