[node-patches] Change in ovirt-node[master]: firewall: Fix plain iptables logic

fabiand at fedoraproject.org fabiand at fedoraproject.org
Thu Sep 19 09:09:10 UTC 2013


Fabian Deutsch has submitted this change and it was merged.

Change subject: firewall: Fix plain iptables logic
......................................................................


firewall: Fix plain iptables logic

Previously the port was opened but this changed wasn't saved. This case
lead to a situation where the iptables service got restarted and
overwrote/closed the previously opened ports.
Now the configuration is saved (and persisted) after a port got opened.

Change-Id: If33424f62c006720a4c553b32af38d2b033579cc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007417
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/firewall.py
1 file changed, 35 insertions(+), 2 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If33424f62c006720a4c553b32af38d2b033579cc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list