Sandro Bonazzola has posted comments on this change.

Change subject: WIP: packaging: setup: host rename
......................................................................


Patch Set 11: (2 inline comments)

....................................................
File packaging/setup/plugins/ovirt-engine-rename/core/database.py
Line 44:     @plugin.event(
Line 45:         stage=plugin.Stages.STAGE_MISC,
Line 46:         after=[
Line 47:             osetupcons.Stages.DB_CONNECTION_AVAILABLE,
Line 48:         ],
latest patches submitted by Alon uses tuple instead of lists
Line 49:     )
Line 50:     def _misc(self):
Line 51:         option = 'RedirectServletReportsPage'
Line 52:         try:


....................................................
File packaging/setup/plugins/ovirt-engine-rename/core/postinstall.py
Line 38: 
Line 39: 
Line 40: @util.export
Line 41: class Plugin(plugin.PluginBase):
Line 42:     """post install config file plugin."""
Have you verified what happens running engine-setup-2 after rename has updated 
the post install config?
Line 43: 
Line 44:     def __init__(self, context):
Line 45:         super(Plugin, self).__init__(context=context)
Line 46: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I28cb0285424236fd3e6907694f6bf1ce6ce3367f
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to