)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6de2a5d9e2bbea12379f1c6b554edc554f4cc5bb","unresolved":true,"context_lines":[{"line_number":12,"context_line":"pattern already used for ``Logical_Switch`` rows, making it easier"},{"line_number":13,"context_line":"to correlate Neutron and OVN resources without relying solely"},{"line_number":14,"context_line":"on ``external_ids`` lookups."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: I49f9d5932cb4e637ac65ea191790182c263fa23f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"27835041_3730f0fb","line":15,"updated":"2026-04-21 12:58:30.000000000","message":"Add LP#2149797","commit_id":"17b9fcb38dbba5bd108ec811f71ff52d34d08c8a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"15f58a16e0430f278a47d0d320818439e059e96a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"pattern already used for ``Logical_Switch`` rows, making it easier"},{"line_number":13,"context_line":"to correlate Neutron and OVN resources without relying solely"},{"line_number":14,"context_line":"on ``external_ids`` lookups."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: I49f9d5932cb4e637ac65ea191790182c263fa23f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c49ee70f_88c08aed","line":15,"in_reply_to":"27835041_3730f0fb","updated":"2026-04-28 12:42:49.000000000","message":"Done","commit_id":"17b9fcb38dbba5bd108ec811f71ff52d34d08c8a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e0d52d15939e51229df699aabffa14bb2ac2180b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bb8e2a65_40ba3035","updated":"2026-05-06 07:28:19.000000000","message":"This patch depends on https://review.opendev.org/c/openstack/ovsdbapp/+/987452","commit_id":"7f94d042e2e9fd25d6929328fe72de1a4e339211"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"0c0b5e912ee10b7d68b7fddd2de7e28b267550fc","unresolved":true,"context_lines":[{"line_number":2101,"context_line":"    def update_nat_rules(self, context, router_id, enable_snat, cidrs\u003dNone,"},{"line_number":2102,"context_line":"                         txn\u003dNone):"},{"line_number":2103,"context_line":"        if enable_snat:"},{"line_number":2104,"context_line":"            idl_func \u003d self._nb_idl.add_nat_rule_in_lrouter"},{"line_number":2105,"context_line":"        else:"},{"line_number":2106,"context_line":"            idl_func \u003d self._nb_idl.delete_nat_rule_in_lrouter"},{"line_number":2107,"context_line":"        func \u003d functools.partial("}],"source_content_type":"text/x-python","patch_set":1,"id":"3563a04b_f78b935f","line":2104,"updated":"2026-04-20 13:43:42.000000000","message":"Do we need to handle this as well if we are changing the signature to take nat_uuid?","commit_id":"17b9fcb38dbba5bd108ec811f71ff52d34d08c8a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"168f0db53b2ed18ac228825cc77fff074ad97a8e","unresolved":false,"context_lines":[{"line_number":2101,"context_line":"    def update_nat_rules(self, context, router_id, enable_snat, cidrs\u003dNone,"},{"line_number":2102,"context_line":"                         txn\u003dNone):"},{"line_number":2103,"context_line":"        if enable_snat:"},{"line_number":2104,"context_line":"            idl_func \u003d self._nb_idl.add_nat_rule_in_lrouter"},{"line_number":2105,"context_line":"        else:"},{"line_number":2106,"context_line":"            idl_func \u003d self._nb_idl.delete_nat_rule_in_lrouter"},{"line_number":2107,"context_line":"        func \u003d functools.partial("}],"source_content_type":"text/x-python","patch_set":1,"id":"bfa2d8be_62a50214","line":2104,"in_reply_to":"3563a04b_f78b935f","updated":"2026-04-21 11:15:14.000000000","message":"Right! Thanks for catching this future bug","commit_id":"17b9fcb38dbba5bd108ec811f71ff52d34d08c8a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"15f58a16e0430f278a47d0d320818439e059e96a","unresolved":false,"context_lines":[{"line_number":2101,"context_line":"    def update_nat_rules(self, context, router_id, enable_snat, cidrs\u003dNone,"},{"line_number":2102,"context_line":"                         txn\u003dNone):"},{"line_number":2103,"context_line":"        if enable_snat:"},{"line_number":2104,"context_line":"            idl_func \u003d self._nb_idl.add_nat_rule_in_lrouter"},{"line_number":2105,"context_line":"        else:"},{"line_number":2106,"context_line":"            idl_func \u003d self._nb_idl.delete_nat_rule_in_lrouter"},{"line_number":2107,"context_line":"        func \u003d functools.partial("}],"source_content_type":"text/x-python","patch_set":1,"id":"7dd6d111_e36fda98","line":2104,"in_reply_to":"bfa2d8be_62a50214","updated":"2026-04-28 12:42:49.000000000","message":"Actually this patch is only for the FIP related `NAT` entries. This one in particular is for the type\u003d\u0027snat\u0027 rules, related to a router.","commit_id":"17b9fcb38dbba5bd108ec811f71ff52d34d08c8a"}]}