)]}'
{"ovsdbapp/schema/ovn_northbound/commands.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e831df0dac52647b1379b431a0b0512d687e3061","unresolved":true,"context_lines":[{"line_number":907,"context_line":"            if self.prefix \u003d\u003d route.ip_prefix:"},{"line_number":908,"context_line":"                lr.delvalue(\u0027static_routes\u0027, route)"},{"line_number":909,"context_line":"                # There should only be one possible match"},{"line_number":910,"context_line":"                route.delete()"},{"line_number":911,"context_line":"                return"},{"line_number":912,"context_line":""},{"line_number":913,"context_line":"        if not self.if_exists:"}],"source_content_type":"text/x-python","patch_set":1,"id":"834a753f_71fc8dcb","line":910,"range":{"start_line":910,"start_character":28,"end_line":910,"end_character":30},"updated":"2021-09-21 14:02:40.000000000","message":"I don\u0027t understand this: route is an element of \"lr.static_routes\". We are removing it in L908. We don\u0027t need this call. What I am missing here?","commit_id":"955656e26379d380419d8c22b1840670869bae9d"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"cba486a8a903d8224668794d35105012b230ad7a","unresolved":true,"context_lines":[{"line_number":907,"context_line":"            if self.prefix \u003d\u003d route.ip_prefix:"},{"line_number":908,"context_line":"                lr.delvalue(\u0027static_routes\u0027, route)"},{"line_number":909,"context_line":"                # There should only be one possible match"},{"line_number":910,"context_line":"                route.delete()"},{"line_number":911,"context_line":"                return"},{"line_number":912,"context_line":""},{"line_number":913,"context_line":"        if not self.if_exists:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9cb3d7a6_52d32f03","line":910,"range":{"start_line":910,"start_character":28,"end_line":910,"end_character":30},"in_reply_to":"834a753f_71fc8dcb","updated":"2021-09-21 15:34:32.000000000","message":"Sorry, this is my fault. When debugging, it seemed to me that the record was being deleted from \u0027lr.static_routes\u0027, but it steel exists in the \u0027Logical_Router_Static_Route\u0027 table. I was guided by LrNatDelCommand.\nNow I double-checked, everything is working correctly, sorry again.","commit_id":"955656e26379d380419d8c22b1840670869bae9d"}]}