commit 9d74de162a2dd7bf5c2df59ccf9ff812f8e46387
Author: Jakub Libosvar <email address hidden>
Date: Mon Oct 9 15:33:32 2017 +0000
ovs-fw: Remove iptables rules on hybrid ports
ovs-firewall now scans ports on its bridge and stores those that have
prefix 'qvo', which means such ports use hybrid plugging. Because
ovs-agent makes a full-sync when it's started, all ports that reside on
the node are passed to firewall driver to refresh firewall, a new helper
was added.
In case the initial scan noticed hybrid plugged, an iptables firewall
driver is instantiated and each port is passed down to helper that
removes iptables rules for given port.
Once all ports are processed, a mark is added to ovsdb to avoid cleaning
iptables in the future. That means next time ovs-agent is started
iptables firewall will not be instantiated.
NOTE: Fullstack tests are a great candidate to cover the migration but
I'll leave it as TODO after we stabilize fullstack tests.
Reviewed: https:/ /review. openstack. org/510628 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=9d74de162a2 dd7bf5c2df59ccf 9ff812f8e46387
Committed: https:/
Submitter: Jenkins
Branch: master
commit 9d74de162a2dd7b f5c2df59ccf9ff8 12f8e46387
Author: Jakub Libosvar <email address hidden>
Date: Mon Oct 9 15:33:32 2017 +0000
ovs-fw: Remove iptables rules on hybrid ports
ovs-firewall now scans ports on its bridge and stores those that have
prefix 'qvo', which means such ports use hybrid plugging. Because
ovs-agent makes a full-sync when it's started, all ports that reside on
the node are passed to firewall driver to refresh firewall, a new helper
was added.
In case the initial scan noticed hybrid plugged, an iptables firewall
driver is instantiated and each port is passed down to helper that
removes iptables rules for given port.
Once all ports are processed, a mark is added to ovsdb to avoid cleaning
iptables in the future. That means next time ovs-agent is started
iptables firewall will not be instantiated.
NOTE: Fullstack tests are a great candidate to cover the migration but
I'll leave it as TODO after we stabilize fullstack tests.
Closes-bug: #1721895
Change-Id: I662c310133a089 bf29b734c539e57 a8cff923074