)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"3f8a78f1a4f4cd49251c8500099db0d7d48cfdaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4cfb2167_6c630797","updated":"2022-01-05 13:06:11.000000000","message":"Hi,\n\nFirst instead of reading all the code I just pulled the change into my devstack to see how it behaves.\n\nThis is my vlan-only environment:\nvim /etc/neutron/plugins/ml2/ml2_conf.ini\n[ml2]\ntenant_network_types \u003d vlan\n[agent]\n# no tunnel_types config option given\n\nWithout this patch ovs-agent did not start. With this patch, ovs-agent starts properly. So far it\u0027s all good.\n\nHowever the mirroring does not seem to work as expected. I only tested between two vms on the same host. After creating a bi-directional tap flow, I only see one direction (OUT) being mirrored:\n\nopenstack tap service create --port port0 --name ts0\nopenstack tap flow create --port port1 --tap-service ts0 --direction BOTH --name tf0\n\n# tcpdump -vvv -n -i ens2 icmp\ntcpdump: listening on ens2, link-type EN10MB (Ethernet), capture size 262144 bytes\n13:03:26.542670 IP (tos 0x0, ttl 64, id 9293, offset 0, flags [DF], proto ICMP (1), length 84)\n    10.0.4.48 \u003e 10.0.4.2: ICMP echo request, id 64513, seq 1781, length 64\n13:03:27.538880 IP (tos 0x0, ttl 64, id 9367, offset 0, flags [DF], proto ICMP (1), length 84)\n    10.0.4.48 \u003e 10.0.4.2: ICMP echo request, id 64513, seq 1782, length 64\n13:03:28.539115 IP (tos 0x0, ttl 64, id 9606, offset 0, flags [DF], proto ICMP (1), length 84)\n    10.0.4.48 \u003e 10.0.4.2: ICMP echo request, id 64513, seq 1783, length 64\n\nIs this supposed to work with this patch, or only if I take the next patch in the series too? On a vxlan network both directions were mirrored as expected.","commit_id":"d2aba037b2d02fdf9d443daed0d4d22ee67ff670"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"6abc00de9ecaf508e9feab92dd5d5b298c187df1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6d72aae5_ecd880c6","in_reply_to":"2c423efa_f51327f6","updated":"2022-01-12 14:59:25.000000000","message":"Thanks, sadly this is what works now I mean not works. The next patch should fix the mirroring, but in multihost env there\u0027s still missing something","commit_id":"d2aba037b2d02fdf9d443daed0d4d22ee67ff670"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"aad6a4eb10fcc28f1e009e4e08140c1d5d6e513d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2c423efa_f51327f6","in_reply_to":"4cfb2167_6c630797","updated":"2022-01-05 13:13:08.000000000","message":"For the record I used the following vxlan and vlan networks:\n\n +---------------------------+----------------------------------------------------------------------------+\n | Field                     | Value                                                                      |\n +---------------------------+----------------------------------------------------------------------------+\n | admin_state_up            | UP                                                                         |\n | availability_zone_hints   |                                                                            |\n | availability_zones        | nova                                                                       |\n | created_at                | 2022-01-04T11:39:57Z                                                       |\n | description               |                                                                            |\n | dns_domain                | None                                                                       |\n | id                        | cb311b8c-994e-4ada-b9e1-a931693b5d0e                                       |\n | ipv4_address_scope        | None                                                                       |\n | ipv6_address_scope        | None                                                                       |\n | is_default                | None                                                                       |\n | is_vlan_transparent       | None                                                                       |\n | mtu                       | 1450                                                                       |\n | name                      | private                                                                    |\n | port_security_enabled     | True                                                                       |\n | project_id                | c25b9498cfe147dda7c93bc900765554                                           |\n | provider:network_type     | vxlan                                                                      |\n | provider:physical_network | None                                                                       |\n | provider:segmentation_id  | 525                                                                        |\n | qos_policy_id             | None                                                                       |\n | revision_number           | 3                                                                          |\n | router:external           | Internal                                                                   |\n | segments                  | None                                                                       |\n | shared                    | False                                                                      |\n | status                    | ACTIVE                                                                     |\n | subnets                   | 42a713d6-8f61-490f-b081-7469bdd8a6a4, a76bccf3-2db1-4835-a5a0-5ae1f10f55a3 |\n | tags                      |                                                                            |\n | updated_at                | 2022-01-04T11:40:03Z                                                       |\n +---------------------------+----------------------------------------------------------------------------+\n\n +---------------------------+--------------------------------------+\n | Field                     | Value                                |\n +---------------------------+--------------------------------------+\n | admin_state_up            | UP                                   |\n | availability_zone_hints   |                                      |\n | availability_zones        | nova                                 |\n | created_at                | 2022-01-04T11:42:55Z                 |\n | description               |                                      |\n | dns_domain                | None                                 |\n | id                        | 4615a0c2-2058-4abb-9630-e789db25846d |\n | ipv4_address_scope        | None                                 |\n | ipv6_address_scope        | None                                 |\n | is_default                | False                                |\n | is_vlan_transparent       | None                                 |\n | mtu                       | 1500                                 |\n | name                      | net-physnet0                         |\n | port_security_enabled     | True                                 |\n | project_id                | c5b99d0a56124325ad0724a89c36b8df     |\n | provider:network_type     | vlan                                 |\n | provider:physical_network | physnet0                             |\n | provider:segmentation_id  | 100                                  |\n | qos_policy_id             | None                                 |\n | revision_number           | 2                                    |\n | router:external           | External                             |\n | segments                  | None                                 |\n | shared                    | False                                |\n | status                    | ACTIVE                               |\n | subnets                   | 77ad5367-4022-4865-847e-cbb7f30c8dfa |\n | tags                      |                                      |\n | updated_at                | 2022-01-04T11:42:59Z                 |\n +---------------------------+--------------------------------------+","commit_id":"d2aba037b2d02fdf9d443daed0d4d22ee67ff670"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cfe1a0b70dc2122ea82b1b1e1f41bbb7a05818f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"cffbb934_49db664f","updated":"2022-02-15 17:45:01.000000000","message":"recheck","commit_id":"9810c0a1c62474e9e539e1d38b72e2f2194c4396"}],"neutron_taas/services/taas/drivers/linux/ovs_taas.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f33fd8404e6853bbe6107f7d17bb6347420d6599","unresolved":true,"context_lines":[{"line_number":47,"context_line":"    def initialize(self):"},{"line_number":48,"context_line":"        self.int_br \u003d self.agent_api.request_int_br()"},{"line_number":49,"context_line":"        self.tun_br \u003d self.agent_api.request_tun_br()"},{"line_number":50,"context_line":"        self.tap_br \u003d OVSBridge_tap_extension(\u0027br-tap\u0027, self.root_helper)"},{"line_number":51,"context_line":"        self.phy_brs \u003d []"},{"line_number":52,"context_line":"        for phy_br in self.agent_api.request_phy_brs():"},{"line_number":53,"context_line":"            self.phy_brs.append(phy_br)"}],"source_content_type":"text/x-python","patch_set":7,"id":"04d3c443_06f166fa","line":50,"range":{"start_line":50,"start_character":8,"end_line":50,"end_character":73},"updated":"2022-01-19 11:56:07.000000000","message":"to use convenience methods from OpenFlowSwitchMixin (https://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/drivers/openvswitch/agent/openflow/native/ofswitch.py#L48 ) taas l2 extension has to be changed an os-ken app.","commit_id":"6c7e8068c68a1ef84df0aaad7dbd305d79241902"}]}