)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":385,"name":"Jason Kölker","email":"jason@koelker.net","username":"jason-koelker"},"change_message_id":"60bbe09306aed427a3681c3e463e7c86fbe40c39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"86916b7e_d5dcab95","updated":"2025-11-12 18:23:42.000000000","message":"LGTM, just two places I *think* we want to return early since we can\u0027t really do anything later anyway.","commit_id":"369df9e82ccadc937e65eeb7e441e911f6133c54"}],"ovn_bgp_agent/drivers/openstack/utils/wire.py":[{"author":{"_account_id":385,"name":"Jason Kölker","email":"jason@koelker.net","username":"jason-koelker"},"change_message_id":"60bbe09306aed427a3681c3e463e7c86fbe40c39","unresolved":true,"context_lines":[{"line_number":494,"context_line":"        ext_ofport \u003d ovs.get_device_port_at_ovs(external_nic)"},{"line_number":495,"context_line":"    except agent_exc.PortNotFound:"},{"line_number":496,"context_line":"        LOG.debug(\"External NIC %s on bridge %s has no ofport yet\","},{"line_number":497,"context_line":"                  external_nic, bridge)"},{"line_number":498,"context_line":""},{"line_number":499,"context_line":"    # There should normally be a single patch port."},{"line_number":500,"context_line":"    patch_ports \u003d ovs.get_ovs_patch_ports_by_peer(bridge, \u0027br-bgp\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dd788ca3_76f210b3","line":497,"updated":"2025-11-12 18:23:42.000000000","message":"```suggestion\n                  external_nic, bridge)\n\n        return\n```","commit_id":"369df9e82ccadc937e65eeb7e441e911f6133c54"},{"author":{"_account_id":385,"name":"Jason Kölker","email":"jason@koelker.net","username":"jason-koelker"},"change_message_id":"60bbe09306aed427a3681c3e463e7c86fbe40c39","unresolved":true,"context_lines":[{"line_number":501,"context_line":"    if not (ext_ofport and patch_ports):"},{"line_number":502,"context_line":"        LOG.warning(\"Skipping ARP flow addition - ext_ofport: %s,\""},{"line_number":503,"context_line":"                    \" patch ports: %s\","},{"line_number":504,"context_line":"                    ext_ofport, patch_ports)"},{"line_number":505,"context_line":"    ovs.ensure_arp_nd_reflector_flows("},{"line_number":506,"context_line":"        bridge, constants.OVS_RULE_COOKIE, mac, ext_ofport,"},{"line_number":507,"context_line":"        patch_ports[0])"}],"source_content_type":"text/x-python","patch_set":1,"id":"91e2a96f_6c6ced63","line":504,"updated":"2025-11-12 18:23:42.000000000","message":"```suggestion\n                    ext_ofport, patch_ports)\n\n        return\n```","commit_id":"369df9e82ccadc937e65eeb7e441e911f6133c54"}]}