)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"afdea20c7191561b2b51703ec120c3dad1bd3e92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"efe4931e_c9a7ac89","updated":"2025-02-04 22:08:45.000000000","message":"Looks like removal of linuxbridge is causing issues as well","commit_id":"77c64c495dd5543812a752d084142d3f86c7a426"}],"networking_bagpipe/bagpipe_bgp/vpn/ipvpn/mpls_ovs_dataplane.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"afdea20c7191561b2b51703ec120c3dad1bd3e92","unresolved":true,"context_lines":[{"line_number":884,"context_line":"        cfg.StrOpt(\"ovs_bridge\", default\u003d\"br-mpls\", advanced\u003dTrue,"},{"line_number":885,"context_line":"                   help\u003d(\"Name of the OVS bridge to use, this has to be the \""},{"line_number":886,"context_line":"                         \"same as the tunneling bridge of the Neutron OVS \""},{"line_number":887,"context_line":"                         \"agent, default is br-mpls\")),"},{"line_number":888,"context_line":"        cfg.IntOpt(\"input_table\", default\u003d0, advanced\u003dTrue,"},{"line_number":889,"context_line":"                   help\u003d(\"specify which OVS table will host the rules \""},{"line_number":890,"context_line":"                         \"for traffic from VRFs\")),"}],"source_content_type":"text/x-python","patch_set":2,"id":"63ac78ae_ceefabd2","line":887,"range":{"start_line":887,"start_character":33,"end_line":887,"end_character":51},"updated":"2025-02-04 22:08:45.000000000","message":"default text shouldn\u0027t be necessary with default\u003d value above","commit_id":"77c64c495dd5543812a752d084142d3f86c7a426"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"afdea20c7191561b2b51703ec120c3dad1bd3e92","unresolved":true,"context_lines":[{"line_number":886,"context_line":"                         \"same as the tunneling bridge of the Neutron OVS \""},{"line_number":887,"context_line":"                         \"agent, default is br-mpls\")),"},{"line_number":888,"context_line":"        cfg.IntOpt(\"input_table\", default\u003d0, advanced\u003dTrue,"},{"line_number":889,"context_line":"                   help\u003d(\"specify which OVS table will host the rules \""},{"line_number":890,"context_line":"                         \"for traffic from VRFs\")),"},{"line_number":891,"context_line":"        cfg.IntOpt(\"ovs_table_id_start\", default\u003d1, advanced\u003dTrue,"},{"line_number":892,"context_line":"                   help\u003d(\"OVS table to use for encap_in, must be different \""}],"source_content_type":"text/x-python","patch_set":2,"id":"10263f3b_4a327305","line":889,"range":{"start_line":889,"start_character":26,"end_line":889,"end_character":33},"updated":"2025-02-04 22:08:45.000000000","message":"s/Specifies","commit_id":"77c64c495dd5543812a752d084142d3f86c7a426"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"afdea20c7191561b2b51703ec120c3dad1bd3e92","unresolved":true,"context_lines":[{"line_number":891,"context_line":"        cfg.IntOpt(\"ovs_table_id_start\", default\u003d1, advanced\u003dTrue,"},{"line_number":892,"context_line":"                   help\u003d(\"OVS table to use for encap_in, must be different \""},{"line_number":893,"context_line":"                         \"input_table! The vrf table number will be the \""},{"line_number":894,"context_line":"                         \"following one.\")),"},{"line_number":895,"context_line":"        cfg.StrOpt(\"gre_tunnel\", default\u003d\"mpls_gre\", advanced\u003dTrue,"},{"line_number":896,"context_line":"                   help\u003d\"OVS interface name for MPLS/GRE encap\"),"},{"line_number":897,"context_line":"        cfg.ListOpt(\"gre_tunnel_options\", default\u003d[],"}],"source_content_type":"text/x-python","patch_set":2,"id":"47ca550f_4cd0621d","line":894,"updated":"2025-02-04 22:08:45.000000000","message":"Maybe re-word this to be:\n\n\"The starting OVS table number to use for additional tables. The values for the encap-in and VRF tables are derived from this. NOTE: This must be different than input_table config option.\"","commit_id":"77c64c495dd5543812a752d084142d3f86c7a426"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"afdea20c7191561b2b51703ec120c3dad1bd3e92","unresolved":true,"context_lines":[{"line_number":902,"context_line":"                          \"interface options (e.g. \u0027options:packet_type\u003d\""},{"line_number":903,"context_line":"                          \"legacy_l3 options:...\u0027)\")),"},{"line_number":904,"context_line":"        cfg.IntOpt(\"ovsbr_interfaces_mtu\", advanced\u003dTrue,"},{"line_number":905,"context_line":"                   help\u003d(\"MTU used for OVS bridge interface.\")),"},{"line_number":906,"context_line":"        cfg.StrOpt(\"hash_method\","},{"line_number":907,"context_line":"                   choices\u003d[\"hash\", \"dp_hash\"],"},{"line_number":908,"context_line":"                   default\u003d\"dp_hash\","}],"source_content_type":"text/x-python","patch_set":2,"id":"5636d96e_9e61af5d","line":905,"range":{"start_line":905,"start_character":50,"end_line":905,"end_character":59},"updated":"2025-02-04 22:08:45.000000000","message":"nit: s/interfaces","commit_id":"77c64c495dd5543812a752d084142d3f86c7a426"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"044d35de346eec0413a8093a9cf954a69871c9c6","unresolved":true,"context_lines":[{"line_number":887,"context_line":"                         \"agent.\")),"},{"line_number":888,"context_line":"        cfg.IntOpt(\"input_table\", default\u003d0, advanced\u003dTrue,"},{"line_number":889,"context_line":"                   help\u003d(\"Specifies which OVS table will host the rules \""},{"line_number":890,"context_line":"                         \"for traffic from VRFs\")),"},{"line_number":891,"context_line":"        cfg.IntOpt(\"ovs_table_id_start\", default\u003d1, advanced\u003dTrue,"},{"line_number":892,"context_line":"                   help\u003d(\"The starting OVS table number to use for \""},{"line_number":893,"context_line":"                         \"additional tables. The values for the encap-in \""}],"source_content_type":"text/x-python","patch_set":3,"id":"28d4fd00_52cd545e","line":890,"updated":"2025-02-18 21:34:08.000000000","message":"super-nit: missing period if you have to respin.","commit_id":"f4381ef367ba9dd8d9ce091dcaabb152379e3299"}]}