)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"57f513a38d83ec8a4d884020b1b60d549698cfb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d8858f22_3334035d","updated":"2026-05-06 11:49:44.000000000","message":"Minor nits, related to the documentation.\n\nIf I\u0027m not wrong, this bug only affects master (2026.2), due to (in this order):\n* https://review.opendev.org/c/openstack/neutron/+/947317\n* https://review.opendev.org/c/openstack/neutron/+/985363. This patch is fixing the first one and introducing the error fixed in this bug.","commit_id":"9851d9ee04bd63fdcca823c8963c8eed0f85ddf5"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"cbf905390186c0f35bf41516698d3d7a6c8575ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"463e7aba_fec67c1b","updated":"2026-05-07 10:21:59.000000000","message":"recheck openstacksdk-functional-devstack","commit_id":"299b0c5c883a777d4c101fd73790a004d15e3d75"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"57f513a38d83ec8a4d884020b1b60d549698cfb8","unresolved":true,"context_lines":[{"line_number":952,"context_line":"        # Only set gateway_port on NAT rules when distributed floating"},{"line_number":953,"context_line":"        # IPs are enabled. Without it, routers get pinned to a chassis"},{"line_number":954,"context_line":"        # (LR.options.chassis), making northd classify them as L3"},{"line_number":955,"context_line":"        # gateway routers that reject gateway_port LP#2150866."},{"line_number":956,"context_line":"        if ovn_conf.is_ovn_distributed_floating_ip():"},{"line_number":957,"context_line":"            router_db \u003d self._l3_plugin.get_router(admin_context, router_id)"},{"line_number":958,"context_line":"            gw_port_id \u003d router_db.get(\u0027gw_port_id\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"4f253918_933eb6a2","line":955,"range":{"start_line":955,"start_character":51,"end_line":955,"end_character":61},"updated":"2026-05-06 11:49:44.000000000","message":"There is a lot of history in this code. I would suggest to provide it. Something like:\n```\nSee, in the following order, LP#2035281, LP#2083527 and LP#2150866.\n```","commit_id":"9851d9ee04bd63fdcca823c8963c8eed0f85ddf5"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"ed521fc2e52d96bd7a76a2501028fb7ac39b1102","unresolved":false,"context_lines":[{"line_number":952,"context_line":"        # Only set gateway_port on NAT rules when distributed floating"},{"line_number":953,"context_line":"        # IPs are enabled. Without it, routers get pinned to a chassis"},{"line_number":954,"context_line":"        # (LR.options.chassis), making northd classify them as L3"},{"line_number":955,"context_line":"        # gateway routers that reject gateway_port LP#2150866."},{"line_number":956,"context_line":"        if ovn_conf.is_ovn_distributed_floating_ip():"},{"line_number":957,"context_line":"            router_db \u003d self._l3_plugin.get_router(admin_context, router_id)"},{"line_number":958,"context_line":"            gw_port_id \u003d router_db.get(\u0027gw_port_id\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"e197e0a6_eb563e2e","line":955,"range":{"start_line":955,"start_character":51,"end_line":955,"end_character":61},"in_reply_to":"4f253918_933eb6a2","updated":"2026-05-06 13:26:25.000000000","message":"Done","commit_id":"9851d9ee04bd63fdcca823c8963c8eed0f85ddf5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"57f513a38d83ec8a4d884020b1b60d549698cfb8","unresolved":true,"context_lines":[{"line_number":957,"context_line":"            router_db \u003d self._l3_plugin.get_router(admin_context, router_id)"},{"line_number":958,"context_line":"            gw_port_id \u003d router_db.get(\u0027gw_port_id\u0027)"},{"line_number":959,"context_line":"            lrp \u003d self._nb_idl.get_lrouter_port(gw_port_id)"},{"line_number":960,"context_line":"            if lrp and lrp.ha_chassis_group:"},{"line_number":961,"context_line":"                columns[\u0027gateway_port\u0027] \u003d lrp.uuid"},{"line_number":962,"context_line":""},{"line_number":963,"context_line":"        if ovn_conf.is_ovn_distributed_floating_ip():"}],"source_content_type":"text/x-python","patch_set":4,"id":"fed472cd_d1c13de8","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":44},"updated":"2026-05-06 11:49:44.000000000","message":"Please, just to remember why this check, add a comment before this line. Something like:\n```\nIf the gateway LRP is scheduled on a chassis (it has ha_chassis_group),\nthen assign the gateway_port reference.\n```","commit_id":"9851d9ee04bd63fdcca823c8963c8eed0f85ddf5"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"ed521fc2e52d96bd7a76a2501028fb7ac39b1102","unresolved":false,"context_lines":[{"line_number":957,"context_line":"            router_db \u003d self._l3_plugin.get_router(admin_context, router_id)"},{"line_number":958,"context_line":"            gw_port_id \u003d router_db.get(\u0027gw_port_id\u0027)"},{"line_number":959,"context_line":"            lrp \u003d self._nb_idl.get_lrouter_port(gw_port_id)"},{"line_number":960,"context_line":"            if lrp and lrp.ha_chassis_group:"},{"line_number":961,"context_line":"                columns[\u0027gateway_port\u0027] \u003d lrp.uuid"},{"line_number":962,"context_line":""},{"line_number":963,"context_line":"        if ovn_conf.is_ovn_distributed_floating_ip():"}],"source_content_type":"text/x-python","patch_set":4,"id":"0463e0d5_8aad8b6b","line":960,"range":{"start_line":960,"start_character":12,"end_line":960,"end_character":44},"in_reply_to":"fed472cd_d1c13de8","updated":"2026-05-06 13:26:25.000000000","message":"Done","commit_id":"9851d9ee04bd63fdcca823c8963c8eed0f85ddf5"}]}