Yedidyah Bar David has posted comments on this change. Change subject: packaging: setup: update firewall for all services ......................................................................
Patch Set 7: (1 comment) .................................................... File packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py Line 168: active_managers.append(manager) Line 169: Line 170: if ( Line 171: self.environment[osetupcons.ConfigEnv.UPDATE_FIREWALL] is None and Line 172: managers I wrote a long reply and gerrit somehow dropped it. Perhaps because it was too long or too stupid. In short, I do not understand what you want. Why user can't choose to keep the active manager? Do you want to allow selecting both "I want to update the firewall" and "I want to select None as manager"? Line 173: ): Line 174: self.dialog.note( Line 175: text=_( Line 176: 'Setup can automatically configure the firewall ' -- To view, visit http://gerrit.ovirt.org/20737 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If3c1a634b2e8539ebd604205b5487290c8d8a1a9 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <d...@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-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
