Alon Bar-Lev has posted comments on this change.
Change subject: packaging: setup: force Apache proxy on upgrade
......................................................................
Patch Set 6: (2 inline comments)
....................................................
File packaging/fedora/setup/engine-upgrade.py
Line 581: if handler.getParam("SSLCertificateKeyFile") ==
'/etc/pki/tls/private/localhost.key':
Line 582: handler.editParam("SSLCertificateKeyFile",
basedefs.FILE_APACHE_PRIVATE_KEY)
Line 583: if handler.getParam("SSLCertificateChainFile") == None:
Line 584: handler.editParam("SSLCertificateChainFile",
basedefs.FILE_APACHE_CA_CRT_SRC)
Line 585: handler.close()
well, anyway is in finally, as you still need to close this if exception, no?
Line 586:
Line 587: def commit(self):
Line 588: if os.path.exists(self.TMPAPACHECONF):
Line 589: shutil.move(self.TMPAPACHECONF,
basedefs.FILE_HTTPD_SSL_CONFIG)
Line 1087: )
Line 1088: handler.open()
Line 1089: fqdn = handler.getParam('ENGINE_FQDN')
Line 1090: proxyEnabled = handler.getParam('ENGINE_PROXY_ENABLED') == 'true'
Line 1091: handler.close()
well, anyway is in finally, as you still need to close this if exception, no?
Line 1092: if proxyEnabled:
Line 1093: logging.debug('oVirt engine Apache proxy was already
enabled')
Line 1094: else:
Line 1095: utils.editEngineSysconfigProtocols(
--
To view, visit http://gerrit.ovirt.org/15051
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib46bcf40acff860cd3e52634ade683d7a8836cf6
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
