)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"71d631fd476fd2ca5da2dd9b7b721d54f2a549e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d7678ec0_3d843445","updated":"2022-03-16 05:15:41.000000000","message":"Looks like this option is only backported till 21.12 branch in ovn, for older one i think applying the option would do nothing.","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"85ee3630a9a11ef63657b77cccad73040865d980","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e88dcb2a_23232c9a","updated":"2022-03-15 15:23:47.000000000","message":"Small localized fix, backport looks good (for all branches), just -1 for the question below","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/commands.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"85ee3630a9a11ef63657b77cccad73040865d980","unresolved":true,"context_lines":[{"line_number":406,"context_line":"        options \u003d {\u0027router-port\u0027: self.lrouter_port}"},{"line_number":407,"context_line":"        if self.is_gw_port:"},{"line_number":408,"context_line":"            options[ovn_const.OVN_GATEWAY_NAT_ADDRESSES_KEY] \u003d \u0027router\u0027"},{"line_number":409,"context_line":"            options[ovn_const.OVN_ROUTER_PORT_EXCLUDE_LB_VIPS_GARP] \u003d \u0027true\u0027"},{"line_number":410,"context_line":"        setattr(port, \u0027options\u0027, options)"},{"line_number":411,"context_line":"        setattr(port, \u0027type\u0027, \u0027router\u0027)"},{"line_number":412,"context_line":"        setattr(port, \u0027addresses\u0027, self.lsp_address)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f3fee23e_77a18abd","line":409,"updated":"2022-03-15 15:23:47.000000000","message":"General backport question, is OVN fine if it does not know the option, will it just ignore it? Mostly if we run an older version","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5946d43615a4cda6d6af42d535294517f61e16e8","unresolved":true,"context_lines":[{"line_number":406,"context_line":"        options \u003d {\u0027router-port\u0027: self.lrouter_port}"},{"line_number":407,"context_line":"        if self.is_gw_port:"},{"line_number":408,"context_line":"            options[ovn_const.OVN_GATEWAY_NAT_ADDRESSES_KEY] \u003d \u0027router\u0027"},{"line_number":409,"context_line":"            options[ovn_const.OVN_ROUTER_PORT_EXCLUDE_LB_VIPS_GARP] \u003d \u0027true\u0027"},{"line_number":410,"context_line":"        setattr(port, \u0027options\u0027, options)"},{"line_number":411,"context_line":"        setattr(port, \u0027type\u0027, \u0027router\u0027)"},{"line_number":412,"context_line":"        setattr(port, \u0027addresses\u0027, self.lsp_address)"}],"source_content_type":"text/x-python","patch_set":1,"id":"18595b41_ec306a09","line":409,"in_reply_to":"33576d7e_32df7482","updated":"2022-03-16 08:46:42.000000000","message":"Yeah it will be ignored if not known by OVN","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"4c0372bbf848359971eb49e5e21d89b98a232b52","unresolved":true,"context_lines":[{"line_number":406,"context_line":"        options \u003d {\u0027router-port\u0027: self.lrouter_port}"},{"line_number":407,"context_line":"        if self.is_gw_port:"},{"line_number":408,"context_line":"            options[ovn_const.OVN_GATEWAY_NAT_ADDRESSES_KEY] \u003d \u0027router\u0027"},{"line_number":409,"context_line":"            options[ovn_const.OVN_ROUTER_PORT_EXCLUDE_LB_VIPS_GARP] \u003d \u0027true\u0027"},{"line_number":410,"context_line":"        setattr(port, \u0027options\u0027, options)"},{"line_number":411,"context_line":"        setattr(port, \u0027type\u0027, \u0027router\u0027)"},{"line_number":412,"context_line":"        setattr(port, \u0027addresses\u0027, self.lsp_address)"}],"source_content_type":"text/x-python","patch_set":1,"id":"33576d7e_32df7482","line":409,"in_reply_to":"73749818_037aa0ca","updated":"2022-03-15 17:01:32.000000000","message":"Thanks, sounds OK for backport then!","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"c487dfb9643093964297a3bc00408e2f957ed76a","unresolved":true,"context_lines":[{"line_number":406,"context_line":"        options \u003d {\u0027router-port\u0027: self.lrouter_port}"},{"line_number":407,"context_line":"        if self.is_gw_port:"},{"line_number":408,"context_line":"            options[ovn_const.OVN_GATEWAY_NAT_ADDRESSES_KEY] \u003d \u0027router\u0027"},{"line_number":409,"context_line":"            options[ovn_const.OVN_ROUTER_PORT_EXCLUDE_LB_VIPS_GARP] \u003d \u0027true\u0027"},{"line_number":410,"context_line":"        setattr(port, \u0027options\u0027, options)"},{"line_number":411,"context_line":"        setattr(port, \u0027type\u0027, \u0027router\u0027)"},{"line_number":412,"context_line":"        setattr(port, \u0027addresses\u0027, self.lsp_address)"}],"source_content_type":"text/x-python","patch_set":1,"id":"73749818_037aa0ca","line":409,"in_reply_to":"f3fee23e_77a18abd","updated":"2022-03-15 15:46:42.000000000","message":"I tested it, and if this option does not exists, it simply does nothing","commit_id":"e61b117b3319188089e0e38727d6eb49c2ad5fec"}]}