)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"de5c59b9d9851474b850ed2f7f07927e5747419b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e8f2c80_fed88f4c","updated":"2025-05-16 05:42:59.000000000","message":"Because [1] was introduced in 2024.1, I guess this patch needs to be backported up to this version.\n\n[1]https://review.opendev.org/c/openstack/neutron/+/883988","commit_id":"a37c2d2a1e27266f51259b33c5b2720f41f00b71"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"5c6397bfd9439cdca5b41b7f0be39334b5b7cccf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ccd0c82c_6e6dd4ac","updated":"2025-05-16 16:05:26.000000000","message":"recheck neutron-functional failure not related to patch","commit_id":"a37c2d2a1e27266f51259b33c5b2720f41f00b71"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"4679b946fa93d0411226648e88ace02517d6dee4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c49c8594_2bce3b9f","updated":"2025-05-16 13:13:14.000000000","message":"recheck neutron-tempest-plugin-ovn timeout unrelated to change","commit_id":"a37c2d2a1e27266f51259b33c5b2720f41f00b71"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"41686d64eb15c229bcf9241ac12eab8a6b79770c","unresolved":true,"context_lines":[{"line_number":565,"context_line":"        db_router_ports \u003d {}"},{"line_number":566,"context_line":"        for router in self.l3_plugin.get_routers(ctx):"},{"line_number":567,"context_line":"            if not utils.is_ovn_provider_router(router):"},{"line_number":568,"context_line":"                continue"},{"line_number":569,"context_line":"            db_routers[router[\u0027id\u0027]] \u003d router"},{"line_number":570,"context_line":"            db_extends[router[\u0027id\u0027]] \u003d {}"},{"line_number":571,"context_line":"            db_extends[router[\u0027id\u0027]][\u0027routes\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"48cb120a_662f4407","line":568,"updated":"2025-05-15 14:22:51.000000000","message":"I was wondering if there was a filter you could pass to get_routers() instead but this works as well.","commit_id":"a37c2d2a1e27266f51259b33c5b2720f41f00b71"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"b9c24bc03913f7770017e191089c0ae092751c0d","unresolved":false,"context_lines":[{"line_number":565,"context_line":"        db_router_ports \u003d {}"},{"line_number":566,"context_line":"        for router in self.l3_plugin.get_routers(ctx):"},{"line_number":567,"context_line":"            if not utils.is_ovn_provider_router(router):"},{"line_number":568,"context_line":"                continue"},{"line_number":569,"context_line":"            db_routers[router[\u0027id\u0027]] \u003d router"},{"line_number":570,"context_line":"            db_extends[router[\u0027id\u0027]] \u003d {}"},{"line_number":571,"context_line":"            db_extends[router[\u0027id\u0027]][\u0027routes\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"dcb8e504_e659c063","line":568,"in_reply_to":"48cb120a_662f4407","updated":"2025-05-15 14:35:29.000000000","message":"It makes sense to fix it here, since we are going over all the routers in the Neutron DB anyways. Thanks for the review","commit_id":"a37c2d2a1e27266f51259b33c5b2720f41f00b71"}]}
