Andrey Gordeev has posted comments on this change.

Change subject: packaging: configuring ovirt-engine on upgrade
......................................................................


Patch Set 4: (1 inline comment)

....................................................
File packaging/fedora/setup/common_utils.py
Line 1673:             # Means ssl mode was set
Line 1674:             dbUrl = dbUrl + 
"?ssl=true&sslfactory=org.postgresql.ssl.NonValidatingFactory"
Line 1675: 
Line 1676:     # Get current configuration
Line 1677:     fqdn, httpPort, httpsPort = getHostParams(basedefs.FILE_WEB_CONF)
Will not work if web-conf.js file will be moved to web-conf.js.rpmsave
Line 1678: 
Line 1679:     # Get encrypted password
Line 1680:     masked_value_set = [dbPassword]
Line 1681:     encryptedPassword = encryptEngineDBPass(password=dbPassword,


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idf596fa9ecaae5ccc6580c19563b21f706d11cfc
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Andrey Gordeev <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to