[node-patches] Change in ovirt-node[master]: Prevent adding iptable rule on every run.

fabiand at redhat.com fabiand at redhat.com
Tue Nov 17 16:07:31 UTC 2015


Fabian Deutsch has submitted this change and it was merged.

Change subject: Prevent adding iptable rule on every run.
......................................................................


Prevent adding iptable rule on every run.

if iptables list is empty a rule will be added to /etc/sysconfig/iptables
We need this rule to be added only once .
Therefor we will load the table before checking
if we need to add the rule.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1221999
Change-Id: Ibe3e80834b9fe817b395793d1a2d02db0c206c4f
Signed-off-by: Tolik Litovsky <tlitovsk at redhat.com>
---
M src/ovirt/node/utils/firewall.py
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Fabian Deutsch: Looks good to me, approved; Passed CI tests
  Douglas Schilling Landgraf: Looks good to me, but someone else must approve
  Anatoly Litovsky: Verified



-- 
To view, visit https://gerrit.ovirt.org/48015
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe3e80834b9fe817b395793d1a2d02db0c206c4f
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the node-patches mailing list