)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"e9c1e7e65b795ec710e2ef76fcdccaf3f64fd8d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff4218ad_9f58c96c","updated":"2022-02-04 08:48:23.000000000","message":"Seems we were working on similar issue. I should have check this before as I opened https://bugs.launchpad.net/neutron/+bug/1959903 which looks exactly the same.\n\nI created an slightly different fix for it https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827670. The approach is slightly different, I\u0027m ensuring that we only look for the type of ports we really need (DEVICE_OWNER_ROUTER_INTF), which cannot have 2 ips (ipv4 and ipv6). Perhaps we can add in that patch also this protection in case that is possible in the future.\n\nThere is one important difference. In my patch I ensure when the members are added, the lr is also considered and the loadbalancer added to it. Otherwise, the loadbalancer will go to ACTIVE but there will be no connectivity to it.\n\nAdding -1 to ensure discussion about how to proceed","commit_id":"38efda1e5ae265f0997919dec9d68ea91e74ba36"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1d804f447aafd8a34ba1cf5def1a42d8e6f32d6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d66f7a66_6f7674ec","updated":"2022-01-28 15:32:13.000000000","message":"Thanks works for me","commit_id":"38efda1e5ae265f0997919dec9d68ea91e74ba36"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"61b6501f9f50bc5f7ef3a2739175d130a96fa50e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a0462d29_3b02e18f","in_reply_to":"d66f7a66_6f7674ec","updated":"2022-02-09 10:33:59.000000000","message":"Yes, right i just tested creating LP, it worked with this patch, but still not able to reach the lb from host.","commit_id":"38efda1e5ae265f0997919dec9d68ea91e74ba36"}],"ovn_octavia_provider/tests/unit/test_helper.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"a422a0abd437aa2fb6005e9478d14a2a8265d177","unresolved":true,"context_lines":[{"line_number":1841,"context_line":"            attrs\u003d{"},{"line_number":1842,"context_line":"                \u0027external_ids\u0027: {"},{"line_number":1843,"context_line":"                    ovn_const.OVN_ROUTER_NAME_EXT_ID_KEY: \u0027router2\u0027,"},{"line_number":1844,"context_line":"                    \u0027neutron:cidrs\u0027: \u00272001:DB8::/64 10.10.10.2/24\u0027},"},{"line_number":1845,"context_line":"                \u0027type\u0027: \u0027router\u0027,"},{"line_number":1846,"context_line":"                \u0027options\u0027: {"},{"line_number":1847,"context_line":"                    \u0027router-port\u0027: \u0027lrp-bar-name\u0027},"}],"source_content_type":"text/x-python","patch_set":1,"id":"2b3565a2_8d8c65ea","line":1844,"range":{"start_line":1844,"start_character":37,"end_line":1844,"end_character":68},"updated":"2022-02-04 09:02:34.000000000","message":"I don\u0027t think this is a valid case. For what I see, for dual stack (ipv4/ipv6) tenant networks, there is a different lrp port for each subnet (logical_switch_port with network:router_interface).\n\nThe actual problem is that in provider networks there are other type of router ports, the ones matching network:router_gateway, and those are the ones that have 2 ips in the case of multiple subnets","commit_id":"38efda1e5ae265f0997919dec9d68ea91e74ba36"}]}