Simone Tiraboschi has posted comments on this change. Change subject: Making engine configuration optional during engine-setup ......................................................................
Patch Set 9: (2 comments) http://gerrit.ovirt.org/#/c/28413/9/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/appmode.py File packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/appmode.py: Line 69: name=osetupcons.Stages.CONFIG_APPLICATION_MODE_AVAILABLE Line 70: ) Line 71: def _customization(self): Line 72: if not self.environment[oenginecons.CoreEnv.ENABLE]: Line 73: self._enabled = False > probably same here and at other places, try to reduce nesting. Done Line 74: else: Line 75: self._enabled = True Line 76: if self.environment[ Line 77: osetupcons.ConfigEnv.APPLICATION_MODE http://gerrit.ovirt.org/#/c/28413/9/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/core/misc.py File packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/core/misc.py: Line 53: ), Line 54: after=( Line 55: osetupcons.Stages.DIALOG_TITLES_S_PRODUCT_OPTIONS, Line 56: ), Line 57: priority=plugin.Stages.PRIORITY_HIGH, > why priority high? Maybe it's enough to keep between DIALOG_TITLES_S_PRODUCT_OPTIONS and DIALOG_TITLES_E_PRODUCT_OPTIONS as for other product options. The question is atomic by itself so I don't think we another start and end couple. I agree that it enough to require just after DIALOG_TITLES_E_PRODUCT_OPTIONS and I'm going to fix it. Line 58: ) Line 59: def _customization(self): Line 60: if self.environment[oenginecons.CoreEnv.ENABLE] is None: Line 61: self.environment[ -- To view, visit http://gerrit.ovirt.org/28413 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7a7fbc9d2abc141bed49eff20549289cba4a4a61 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stira...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stira...@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
