neutron/neutron/agent/linux
Rodolfo Alonso Hernandez aaf87aeb55 [OVS][FW] Remote SG IDs left behind when a SG is removed
When any port in the OVS agent is using a security groups (SG) and
this SG is removed, is marked to be deleted. This deletion process
is done in [1].

The SG deletion process consists on removing any reference of this SG
from the firewall and the SG port map. The firewall removes this SG in
[2].

The information of a SG is stored in:
* ConjIPFlowManager.conj_id_map = ConjIdMap(). This class stores the
  conjunction IDS (conj_ids) in a dictionary using the following keys:

    ConjIdMap.id_map[(sg_id, remote_sg_id, direction, ethertype,
      conj_ids)] = conj_id_XXX

* ConjIPFlowManager.conj_ids is a nested dictionary, built in the
  following way:

    self.conj_ids[vlan_tag][(direction, ethertype)][remote_sg_id] = \
      set([conj_id_1, conj_id_2, ...])

This patch stores all conjuntion IDs generated and assigned to the
tuple (sg_id, remote_sg_id, direction, ethertype). When a SG is
removed, the deletion method will look for this SG in the new storage
variable created, ConjIdMap.id_map_group, and will mark all the
conjuntion IDs related to be removed. That will cleanup those rules
left in the OVS matching:
  action=conjunction(conj_id, 1/2)

[1]118930f03d/neutron/agent/linux/openvswitch_firewall/firewall.py (L731)
[2]118930f03d/neutron/agent/linux/openvswitch_firewall/firewall.py (L399)

Conflicts:
      neutron/tests/unit/agent/linux/openvswitch_firewall/test_firewall.py

Change-Id: I63e446a30cf10e7bcd34a6f0d6ba1711301efcbe
Related-Bug: #1881157
(cherry picked from commit 0eebd002cc)
(cherry picked from commit ed22f7a2ff)
(cherry picked from commit 6615f248e2)
2020-08-21 09:55:56 +00:00
..
openvswitch_firewall [OVS][FW] Remote SG IDs left behind when a SG is removed 2020-08-21 09:55:56 +00:00
__init__.py Empty files should not contain copyright or license 2014-10-20 00:50:32 +00:00
bridge_lib.py Change to use iproute2 instead of brctl 2018-11-14 12:46:37 -05:00
daemon.py remove neutron.common.exceptions 2019-02-01 14:35:00 -07:00
dhcp.py Merge "Increase timeout when waiting for dnsmasq enablement" into stable/stein 2019-11-19 14:07:59 +00:00
dibbler.py use PROVISIONAL_IPV6_PD_PREFIX from neutron-lib 2017-11-14 12:26:43 -07:00
ethtool.py Workaround for TCP checksum issue with ovs-dpdk and veth pair 2020-06-29 13:07:04 -05:00
external_process.py Check dnsmasq process is active when spawned 2019-11-14 11:23:56 +00:00
interface.py Merge "Workaround for TCP checksum issue with ovs-dpdk and veth pair" into stable/stein 2020-07-01 19:59:27 +00:00
ip_conntrack.py Fix pep8 E128 warnings in non-test code 2019-03-12 21:22:33 +00:00
ip_lib.py Merge "Kill all processes running in a namespace before deletion" into stable/stein 2020-05-07 20:35:41 +00:00
ip_link_support.py Make code follow log translation guideline 2017-08-14 02:01:48 +00:00
ip_monitor.py Trivial: Move platform independent modules to common dir 2018-08-14 09:54:01 +03:00
ipset_manager.py [Security] fix allowed-address-pair 0.0.0.0/0 issue 2020-07-31 11:23:01 +00:00
iptables_comments.py Iptables firewall driver adds forward rules for trusted ports 2018-03-06 10:13:44 +01:00
iptables_firewall.py [Security] fix allowed-address-pair 0.0.0.0/0 issue 2020-07-31 11:23:01 +00:00
iptables_manager.py Packets getting lost during SNAT with too many connections 2019-05-20 17:39:47 -04:00
keepalived.py [L3 HA] Add "no_track" option to VIPs in keepalived config 2020-04-23 07:14:53 +00:00
l3_tc_lib.py remove neutron.common.exceptions 2019-02-01 14:35:00 -07:00
pd.py use PROVISIONAL_IPV6_PD_PREFIX from neutron-lib 2017-11-14 12:26:43 -07:00
pd_driver.py use PROVISIONAL_IPV6_PD_PREFIX from neutron-lib 2017-11-14 12:26:43 -07:00
ra.py Add radvd_user config option 2019-10-24 15:54:15 +00:00
tc_lib.py Add TC filter functions implemented with pyroute2 2019-03-09 22:01:57 +00:00
utils.py Fix race condition when getting cmdline 2019-11-20 15:13:37 +00:00
xenapi_root_helper.py Fix all pep8 E129 errors 2018-05-03 13:44:04 +09:00