)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"104a278dfe38dde3ea001772835761447f0da655","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"98776c3c_cade75e7","updated":"2024-02-22 14:45:39.000000000","message":"Thanks for the patch, I think we missed the constants in some places","commit_id":"6b372a4880501a74250637eb478aa8a12900a085"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"1b7659e46e98b34c108c717f6010c9184a2384d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"054fd750_ef0e24df","updated":"2024-02-22 14:53:35.000000000","message":"nice finds!","commit_id":"7243e1c1ab85203498e7b4b4f9be1546754ffb3f"}],"ovn_bgp_agent/drivers/openstack/nb_ovn_bgp_driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"104a278dfe38dde3ea001772835761447f0da655","unresolved":true,"context_lines":[{"line_number":736,"context_line":"        if not self._expose_tenant_networks:"},{"line_number":737,"context_line":"            return True"},{"line_number":738,"context_line":""},{"line_number":739,"context_line":"        if CONF.advertisement_method_tenant_networks \u003d\u003d \u0027subnet\u0027:"},{"line_number":740,"context_line":"            # Fix ips to be the network address, instead of the lrp address"},{"line_number":741,"context_line":"            # so the cleanup will not remove them, since they match what\u0027s"},{"line_number":742,"context_line":"            # in the kernel"}],"source_content_type":"text/x-python","patch_set":2,"id":"c5823c11_f78f4497","line":739,"updated":"2024-02-22 14:45:39.000000000","message":"Should we also use the constant here ?","commit_id":"6b372a4880501a74250637eb478aa8a12900a085"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"1b7659e46e98b34c108c717f6010c9184a2384d1","unresolved":false,"context_lines":[{"line_number":736,"context_line":"        if not self._expose_tenant_networks:"},{"line_number":737,"context_line":"            return True"},{"line_number":738,"context_line":""},{"line_number":739,"context_line":"        if CONF.advertisement_method_tenant_networks \u003d\u003d \u0027subnet\u0027:"},{"line_number":740,"context_line":"            # Fix ips to be the network address, instead of the lrp address"},{"line_number":741,"context_line":"            # so the cleanup will not remove them, since they match what\u0027s"},{"line_number":742,"context_line":"            # in the kernel"}],"source_content_type":"text/x-python","patch_set":2,"id":"7be375df_b9200b6d","line":739,"in_reply_to":"c5823c11_f78f4497","updated":"2024-02-22 14:53:35.000000000","message":"Definitely!","commit_id":"6b372a4880501a74250637eb478aa8a12900a085"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"104a278dfe38dde3ea001772835761447f0da655","unresolved":true,"context_lines":[{"line_number":779,"context_line":"        if not self._expose_tenant_networks:"},{"line_number":780,"context_line":"            return"},{"line_number":781,"context_line":""},{"line_number":782,"context_line":"        if CONF.advertisement_method_tenant_networks \u003d\u003d \u0027subnet\u0027:"},{"line_number":783,"context_line":"            # Fix ips to be the network address, instead of the lrp address"},{"line_number":784,"context_line":"            # so the cleanup will not remove them, since they match what\u0027s"},{"line_number":785,"context_line":"            # in the kernel"}],"source_content_type":"text/x-python","patch_set":2,"id":"b89b5041_49040801","line":782,"updated":"2024-02-22 14:45:39.000000000","message":"ditto","commit_id":"6b372a4880501a74250637eb478aa8a12900a085"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"1b7659e46e98b34c108c717f6010c9184a2384d1","unresolved":false,"context_lines":[{"line_number":779,"context_line":"        if not self._expose_tenant_networks:"},{"line_number":780,"context_line":"            return"},{"line_number":781,"context_line":""},{"line_number":782,"context_line":"        if CONF.advertisement_method_tenant_networks \u003d\u003d \u0027subnet\u0027:"},{"line_number":783,"context_line":"            # Fix ips to be the network address, instead of the lrp address"},{"line_number":784,"context_line":"            # so the cleanup will not remove them, since they match what\u0027s"},{"line_number":785,"context_line":"            # in the kernel"}],"source_content_type":"text/x-python","patch_set":2,"id":"e84ac9bb_b2cfd63e","line":782,"in_reply_to":"b89b5041_49040801","updated":"2024-02-22 14:53:35.000000000","message":"Done","commit_id":"6b372a4880501a74250637eb478aa8a12900a085"}]}