)]}'
{"/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"},{"author":{"_account_id":39125,"name":"Felix Moebius","email":"felix.moebius@digits.schwarz","username":"felixmoebius"},"change_message_id":"5d8483e28f7fca77f4f869efc8b1e72ff72c2647","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":"2148be84_2513ec13","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":7},"in_reply_to":"7c66d3c2_e5581a91","updated":"2026-05-26 07:23:27.000000000","message":"This is one way that can result in an orphaned nat entry. I am not confident that this is the only one, which is why I did not mark this as closing the bug. I can still change it if you prefer though.","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":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9beb804494288e5d43c351d995095c89e3d8e350","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":"249a3722_b95951d9","line":1253,"in_reply_to":"b38daab7_bdbfe119","updated":"2026-06-01 09:05:28.000000000","message":"yeah, I agree with Brian that it would be better to make it debug, or maybe \"warning\" which would give some information to the cloud operator but still don\u0027t say it is really an issue in the cloud (as it seems it is not)","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"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"823cab278af52d8c67592c43315ab6d6406ac7d8","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":"8ad6b284_cc5a690d","line":1253,"range":{"start_line":1253,"start_character":16,"end_line":1253,"end_character":21},"in_reply_to":"88311e09_90eb6217","updated":"2026-05-26 21:44:33.000000000","message":"I\u0027m fine changing it back based on Rodolfo\u0027s comment.","commit_id":"723bd3bcd7b16136a0ddf497110f9b94b6cdc26a"},{"author":{"_account_id":39125,"name":"Felix Moebius","email":"felix.moebius@digits.schwarz","username":"felixmoebius"},"change_message_id":"5d8483e28f7fca77f4f869efc8b1e72ff72c2647","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":"88311e09_90eb6217","line":1253,"range":{"start_line":1253,"start_character":16,"end_line":1253,"end_character":21},"in_reply_to":"92a4f513_3cf72f0b","updated":"2026-05-26 07:23:27.000000000","message":"I changed this to debug log level at Brian\u0027s request. I am fine with either.","commit_id":"723bd3bcd7b16136a0ddf497110f9b94b6cdc26a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7e4293ef9c3ce18c74501121cf3704d8f671708d","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.warning(\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":5,"id":"7040153c_e4d86432","line":1253,"range":{"start_line":1253,"start_character":51,"end_line":1253,"end_character":53},"updated":"2026-06-16 06:59:36.000000000","message":"s/ip/IP","commit_id":"7a70217d78bd7f90b8c9de4d9341715842d2ee12"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7e4293ef9c3ce18c74501121cf3704d8f671708d","unresolved":true,"context_lines":[{"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.warning(\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):"},{"line_number":1257,"context_line":"        lrouter \u003d utils.ovn_name(router_id)"}],"source_content_type":"text/x-python","patch_set":5,"id":"cb6d057d_f898909e","line":1254,"range":{"start_line":1254,"start_character":34,"end_line":1254,"end_character":36},"updated":"2026-06-16 06:59:36.000000000","message":"s/ip/IP","commit_id":"7a70217d78bd7f90b8c9de4d9341715842d2ee12"}]}