Alon Bar-Lev has posted comments on this change.

Change subject: packaging: setup: modify AIA port when upgrade from legacy
......................................................................


Patch Set 1:

(2 comments)

....................................................
Commit Message
Line 10: web server we need to modify AIA extension URL.
Line 11: 
Line 12: this will not effect past certificate, only new certificate.
Line 13: 
Line 14: the engine rename script may be used to refresh the engine certificate.
we leave the jboss http port opened, so nothing will be broken.

I think the instructions should go into documentation.

But I will consider adding a note.
Line 15: 
Line 16: Change-Id: I6d955b34497e3256528180938dc67575d7812646


....................................................
File packaging/setup/plugins/ovirt-engine-setup/legacy/ca.py
Line 68:             with open(name, 'r') as f:
Line 69:                 
self.environment[otopicons.CoreEnv.MAIN_TRANSACTION].append(
Line 70:                     filetransaction.FileTransaction(
Line 71:                         name=name,
Line 72:                         content=f.read().replace(
Well, as this code is temporary, I do not think we should invest more than a 
'good enough' solution.
Line 73:                             replace['from'], replace['to']
Line 74:                         ).splitlines(),
Line 75:                         modifiedList=self.environment[
Line 76:                             otopicons.CoreEnv.MODIFIED_FILES


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6d955b34497e3256528180938dc67575d7812646
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to