Juan Hernandez has posted comments on this change.

Change subject: packaging: Application Mode option in installer
......................................................................


Patch Set 4: (1 inline comment)

....................................................
File packaging/fedora/setup/engine-setup.py
Line 979:                                
output_messages.ERR_DB_SET_SERVICE_TYPE)
Line 980: 
Line 981:     # No change with respect to default application mode, No DB 
update required
Line 982:     if controller.CONF["APPLICATION_MODE"].upper() == "BOTH":
Line 983:         return
Ok, understood, thanks for explaining.

The upgrade scripts can't be modified, as we save a MD5 digest of their 
contents.
Line 984: 
Line 985:     applicationMode = 
controller.CONF["APPLICATION_MODE_ENUM"].parse(str.upper(controller.CONF["APPLICATION_MODE"]))
Line 986: 
Line 987:     sqlQuery = "select fn_db_update_config_value('ApplicationMode', 
'%s', 'general')" % applicationMode


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ieba515d2baf3559bf27185c7c6432160551ef172
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <[email protected]>
Gerrit-Reviewer: Aravinda VK <[email protected]>
Gerrit-Reviewer: Bala.FA <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Ohad Basan <[email protected]>
Gerrit-Reviewer: Ronen Angluster <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to