)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2d2057b27e2b1120e08884cd498277c0171c82fd","unresolved":true,"context_lines":[{"line_number":19,"context_line":"Keep the ovn revision entry in case we don\u0027t find a matching nat entry to"},{"line_number":20,"context_line":"give the maintenance task a chance to retry the deletion."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Related-Bug: #1987530"},{"line_number":23,"context_line":"Signed-off-by: Felix Moebius \u003cfelix.moebius@digits.schwarz\u003e"},{"line_number":24,"context_line":"Change-Id: I6c598cbbca1e449cf67314d022f101f2f73cf1bc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7c66d3c2_e5581a91","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":7},"updated":"2026-05-21 09:52:17.000000000","message":"This is closing the bug. This should be `Closes-Bug: xxx`","commit_id":"723bd3bcd7b16136a0ddf497110f9b94b6cdc26a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8f2b49084885a7134c1c32a129521abeb60feb5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cb20233c_18b496ab","updated":"2026-05-14 00:44:35.000000000","message":"I guess this fixes the scenario you mention, but can you add a unit test? Thanks","commit_id":"7d73e599dbf042b1c018996a229983cf23590369"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9ad05b8d9f9b47fe235206f819bcb70294792f98","unresolved":true,"context_lines":[{"line_number":1250,"context_line":"                              \u0027router. Error: %s\u0027, e)"},{"line_number":1251,"context_line":"            db_rev.delete_revision(context, fip_id, ovn_const.TYPE_FLOATINGIPS)"},{"line_number":1252,"context_line":"        else:"},{"line_number":1253,"context_line":"            LOG.error(\u0027Unable to delete floating ip in gateway router. \u0027"},{"line_number":1254,"context_line":"                      \u0027Floating ip %s not found.\u0027, fip_id)"},{"line_number":1255,"context_line":""},{"line_number":1256,"context_line":"    def disassociate_floatingip(self, context, floatingip, router_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"d72b2890_9899b6a2","line":1253,"updated":"2026-05-20 14:12:19.000000000","message":"Sorry I didn\u0027t notice this before, but this should be LOG.debug as an error didn\u0027t occur and matches other places in this code.","commit_id":"e72a8c4605e9ad600e2d108c688bc2afdd5e5ead"},{"author":{"_account_id":39125,"name":"Felix Moebius","email":"felix.moebius@digits.schwarz","username":"felixmoebius"},"change_message_id":"166ed0efa1cc5ddf7d2a04a7f51c0240d0827c97","unresolved":true,"context_lines":[{"line_number":1250,"context_line":"                              \u0027router. Error: %s\u0027, e)"},{"line_number":1251,"context_line":"            db_rev.delete_revision(context, fip_id, ovn_const.TYPE_FLOATINGIPS)"},{"line_number":1252,"context_line":"        else:"},{"line_number":1253,"context_line":"            LOG.error(\u0027Unable to delete floating ip in gateway router. \u0027"},{"line_number":1254,"context_line":"                      \u0027Floating ip %s not found.\u0027, fip_id)"},{"line_number":1255,"context_line":""},{"line_number":1256,"context_line":"    def disassociate_floatingip(self, context, floatingip, router_id):"}],"source_content_type":"text/x-python","patch_set":2,"id":"b38daab7_bdbfe119","line":1253,"in_reply_to":"d72b2890_9899b6a2","updated":"2026-05-20 15:03:05.000000000","message":"All good, my thinking was that it might be useful to see these kind of things in logs if it occurs regularly as it indicates that load on the northbound is higher than what the idl is able to keep up with. But debug log level is also fine with me if you think that makes more sense.","commit_id":"e72a8c4605e9ad600e2d108c688bc2afdd5e5ead"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"42725f3a894c43264cc75ac977aa785e8fdc2abd","unresolved":true,"context_lines":[{"line_number":1250,"context_line":"                              \u0027router. Error: %s\u0027, e)"},{"line_number":1251,"context_line":"            db_rev.delete_revision(context, fip_id, ovn_const.TYPE_FLOATINGIPS)"},{"line_number":1252,"context_line":"        else:"},{"line_number":1253,"context_line":"            LOG.debug(\u0027Unable to delete floating ip in gateway router. \u0027"},{"line_number":1254,"context_line":"                      \u0027Floating ip %s not found.\u0027, fip_id)"},{"line_number":1255,"context_line":""},{"line_number":1256,"context_line":"    def disassociate_floatingip(self, context, floatingip, router_id):"}],"source_content_type":"text/x-python","patch_set":3,"id":"92a4f513_3cf72f0b","line":1253,"range":{"start_line":1253,"start_character":16,"end_line":1253,"end_character":21},"updated":"2026-05-21 09:51:45.000000000","message":"This should be an error, the NAT register should be present. If we delete the Neutron DB FIP but not the OVN NAT register, we are leaving a leftover in the OVN DB (NAT register) and in the Neutron DB (OVNRevisionNumbers register)","commit_id":"723bd3bcd7b16136a0ddf497110f9b94b6cdc26a"}]}