neutron/neutron/tests/unit/agent/linux
Rodolfo Alonso Hernandez 6615f248e2 [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)
2020-07-31 16:05:11 +00:00
..
openvswitch_firewall [OVS][FW] Remote SG IDs left behind when a SG is removed 2020-07-31 16:05:11 +00:00
__init__.py Empty files should not contain copyright or license 2014-10-20 00:50:32 +00:00
failing_process.py Keep reading stdout/stderr until after kill 2015-11-19 19:14:13 +02:00
test_bridge_lib.py Change to use iproute2 instead of brctl 2018-11-14 12:46:37 -05:00
test_daemon.py use test tools from neutron-lib 2019-06-05 06:21:04 -06:00
test_dhcp.py Fixes dnsmasq host file parsing with "addr6_list" 2020-06-22 15:18:37 +00:00
test_external_process.py Check dnsmasq process is active when spawned 2019-10-30 12:23:56 +00:00
test_interface.py Merge "Workaround for TCP checksum issue with ovs-dpdk and veth pair" into stable/train 2020-07-01 03:10:18 +00:00
test_ip_conntrack.py Move conntrack zones to IPTablesFirewall 2017-03-30 14:54:51 -07:00
test_ip_lib.py Avoid raising NetworkInterfaceNotFound exception in DHCP agent logs 2020-04-23 15:49:24 +00:00
test_ipset_manager.py Create ipset set_name_exists() method 2015-10-14 10:18:44 -04:00
test_iptables_firewall.py Start enforcing E125 flake8 directive 2019-07-19 23:39:41 -04:00
test_iptables_manager.py Do not initialize the iptables nat table in the dhcp-agent 2019-04-24 20:56:03 -04:00
test_keepalived.py [L3 HA] Add "no_track" option to VIPs in keepalived config 2020-04-23 07:14:25 +00:00
test_l3_tc_lib.py Use Pyroute2 "add_tc_qdisc" function in l3_tc_lib 2019-07-05 08:13:07 +00:00
test_pd.py Fix pep8 E128 warnings in non-test code 2019-03-12 21:22:33 +00:00
test_tc_lib.py Fix the wrong value for QoS rate conversion to bytes/s 2020-06-24 17:22:02 +00:00
test_utils.py Fix race condition when getting cmdline 2019-09-18 21:32:33 -04:00
test_xenapi_root_helper.py Use os-xenapi for neutron when XenServer as hypervisor 2017-03-30 18:33:37 +00:00