Change in ovirt-hosted-engine-setup[master]: packaging: firewall: override iptables configuration

sbonazzo at redhat.com sbonazzo at redhat.com
Wed Oct 9 15:33:33 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: firewall: override iptables configuration
......................................................................


packaging: firewall: override iptables configuration

Previously iptables configuration was not overridden when the
host was added to the engine.
Now override_iptables is specified when adding the host.
Requiring iptables also on systems with FirewallD because the
engine still doesn't support FirewallD.

Change-Id: I03b0bc75dcbdc92d6f430e3a11d0549a3f0172c0
Bug-Url: https://bugzilla.redhat.com/1014616
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M ovirt-hosted-engine-setup.spec.in
M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
M src/plugins/ovirt-hosted-engine-setup/network/firewall_manager.py
3 files changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Yedidyah Bar David: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I03b0bc75dcbdc92d6f430e3a11d0549a3f0172c0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov at gmail.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list