Change in vdsm[master]: tests: don't fail with firewalld if a veth is gone already

osvoboda at redhat.com osvoboda at redhat.com
Sun May 8 09:17:45 UTC 2016


Ondřej Svoboda has uploaded a new change for review.

Change subject: tests: don't fail with firewalld if a veth is gone already
......................................................................

tests: don't fail with firewalld if a veth is gone already

Change-Id: I02b2651cba09e74321e9b9ad658979798eb4458a
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
---
M tests/network/firewall.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/75/57175/1

diff --git a/tests/network/firewall.py b/tests/network/firewall.py
index b288c9d..c11c731 100644
--- a/tests/network/firewall.py
+++ b/tests/network/firewall.py
@@ -93,8 +93,8 @@
                         iface, '-p', 'udp', '--sport', '546', '--dport',
                         '547', '-j', 'ACCEPT'])  # DHCPv6
     elif _serviceRunning('firewalld'):
-        _execCmdChecker([_FIREWALLD_BINARY.cmd, '--zone=trusted',
-                        '--remove-interface=' + iface])
+        execCmd([_FIREWALLD_BINARY.cmd, '--zone=trusted',
+                 '--remove-interface=' + iface])
     else:
         logging.warning('No firewall service detected.')
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I02b2651cba09e74321e9b9ad658979798eb4458a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>


More information about the vdsm-patches mailing list