)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4a5632d5e240cda02e4019efe48eab9033ccdb35","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     alisafari \u003calisafar1212@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-02-23 09:12:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix lb association to routers process"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #2052843"},{"line_number":10,"context_line":"Change-Id: Icc66011f671fa72e8c4192ed7ede4d562177fe84"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ca39da63_2d678508","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2024-02-27 09:04:40.000000000","message":"Explain a bit more, in the description, what this patch is doing. That will help to review it and, in a future, to track the git history.","commit_id":"101ff931e166278c70b2a784d5e6e9d7420581f3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9c08465e293afa9476c55f1fedcb082fd52e0133","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"36bcee65_badb4193","updated":"2026-03-13 10:38:33.000000000","message":"Hi, I\u0027ve reused part of your patch in a new review at https://review.opendev.org/c/openstack/ovn-octavia-provider/+/980407 (didn\u0027t want to override your changes here)","commit_id":"28179bda189cec89ead73d062b976eded62fc521"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"dd6cefb4479c388297194c316afd48b39bdfea01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c69a56e6_cc0050ae","updated":"2026-03-12 12:30:35.000000000","message":"I doesn\u0027t work well when a router that includes a subnet already plugged into another router is deleted\nthe subnet is disassociated from the LB while still connected through the other router","commit_id":"28179bda189cec89ead73d062b976eded62fc521"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"a5f34b4a567bf507ee0ddd0ab00b6234c12cd1f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d86edc66_4c2f9666","updated":"2026-03-12 12:31:33.000000000","message":"it also needs to support the \"sync\" feature of ovn-provider (which was probably introduced after this patch)","commit_id":"28179bda189cec89ead73d062b976eded62fc521"}],"ovn_octavia_provider/helper.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4a5632d5e240cda02e4019efe48eab9033ccdb35","unresolved":true,"context_lines":[{"line_number":843,"context_line":"                        continue"},{"line_number":844,"context_line":"                lsp_router_ports.append(port)"},{"line_number":845,"context_line":"        result \u003d []"},{"line_number":846,"context_line":"        for lsp in lsp_router_ports:"},{"line_number":847,"context_line":"            lr \u003d self._find_lr_from_lsp(lsp)"},{"line_number":848,"context_line":"            if lr is not None:"},{"line_number":849,"context_line":"                result.append(lr)"},{"line_number":850,"context_line":"        return result"},{"line_number":851,"context_line":""},{"line_number":852,"context_line":"    def _find_lr_from_lsp(self, lsp_router_port):"},{"line_number":853,"context_line":"        lrp_name \u003d lsp_router_port.options.get(\u0027router-port\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"608758f5_3cb37de0","line":850,"range":{"start_line":846,"start_character":8,"end_line":850,"end_character":21},"updated":"2024-02-27 09:04:40.000000000","message":"Now we can have multiple GW ports in a router, please check the result doens\u0027t contain a duplicated LR register.","commit_id":"101ff931e166278c70b2a784d5e6e9d7420581f3"}]}
