Change in ovirt-engine[master]: packaging: setup: firewall-manager - check status

[oschreib at redhat.com]

Ofer Schreiber has submitted this change and it was merged.

Change subject: packaging: setup: firewall-manager - check status
......................................................................


packaging: setup: firewall-manager - check status

During setup, ask only about firewall managers that are both enabled
and active.

Currently this means that if both firewalld and iptables are enabled,
but iptables is active and firewalld isn't, first we ask about firewalld,
and if users accepts, we do not even ask about iptables. With this
change, at this state, we only ask about iptables.

Change-Id: Ice741b9738c822d48ab3719dd34cbe989b359afe
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py
1 file changed, 8 insertions(+), 2 deletions(-)

Approvals:
  Ofer Schreiber: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ice741b9738c822d48ab3719dd34cbe989b359afe
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>