Dan Kenigsberg has uploaded a new change for review.
Change subject: libvirt is happy with redefinition of an existing nwfilter
......................................................................
libvirt is happy with redefinition of an existing nwfilter
Change-Id: Id253c66df8296613b6a2bc3fc058ef2754702964
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
---
M lib/vdsm/tool/nwfilter.py
1 file changed, 0 insertions(+), 8 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/88/20788/1
diff --git a/lib/vdsm/tool/nwfilter.py b/lib/vdsm/tool/nwfilter.py
index 54bbe8d..f87f975 100755
--- a/lib/vdsm/tool/nwfilter.py
+++ b/lib/vdsm/tool/nwfilter.py
@@ -56,14 +56,6 @@
define vdsm network filter on libvirt to control VM traffic
"""
- try:
- conn.nwfilterLookupByName(self.filterName).undefine()
- except libvirt.libvirtError:
- # Ignore failure if filter isn't exists or if failed to remove.
- # Failure might occur when attempting to remove a filter which
- # is being used by running VMs
- pass
-
nwFilter = conn.nwfilterDefineXML(self.buildFilterXml())
logging.debug("Filter %s was defined" % nwFilter.name())
--
To view, visit http://gerrit.ovirt.org/20788
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id253c66df8296613b6a2bc3fc058ef2754702964
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
