)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f5df4d7a04b0fd9711005006f21e1f19e7f08d73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"79a6b88f_11e551d7","updated":"2023-05-28 00:55:28.000000000","message":"Opps, a patch was already uploaded","commit_id":"b50664d8c8d32db15a4f42dea026a6a687f98f05"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"78baac61c7d16fbf0f9aa71ecdea2963f05b0d87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"96bda337_934ca950","updated":"2023-05-28 04:08:40.000000000","message":"recheck zone test failed","commit_id":"b50664d8c8d32db15a4f42dea026a6a687f98f05"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f0f7157b71f01dec1adc278994ae80ed806ce87c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d89909b8_647c8ede","updated":"2023-05-31 21:50:15.000000000","message":"Merging so we can backport","commit_id":"5b7ee8899dd041a351ed61a2a6378e75812bfd59"}],"designate/storage/sqlalchemy/base.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"e6823dee798c90e5c7c4c0630e6b88f4378cf740","unresolved":true,"context_lines":[{"line_number":176,"context_line":"    def _apply_tenant_criteria(self, context, table, query,"},{"line_number":177,"context_line":"                               include_null_tenant\u003dTrue,"},{"line_number":178,"context_line":"                               include_shared\u003dFalse):"},{"line_number":179,"context_line":"        _shared_zone_project_id \u003d tables.shared_zones.c.target_project_id"},{"line_number":180,"context_line":"        if hasattr(table.c, \u0027tenant_id\u0027):"},{"line_number":181,"context_line":"            if not context.all_tenants:"},{"line_number":182,"context_line":"                # NOTE: The query doesn\u0027t work with table.c.tenant_id is None,"}],"source_content_type":"text/x-python","patch_set":1,"id":"20fa09bb_916ac4d9","line":179,"updated":"2023-05-26 19:30:39.000000000","message":"nit: I don\u0027t think we need the _ here since it\u0027s within a function","commit_id":"8255772a59b9501402b6e10427d09738f0431c2d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f0f7157b71f01dec1adc278994ae80ed806ce87c","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    def _apply_tenant_criteria(self, context, table, query,"},{"line_number":177,"context_line":"                               include_null_tenant\u003dTrue,"},{"line_number":178,"context_line":"                               include_shared\u003dFalse):"},{"line_number":179,"context_line":"        _shared_zone_project_id \u003d tables.shared_zones.c.target_project_id"},{"line_number":180,"context_line":"        if hasattr(table.c, \u0027tenant_id\u0027):"},{"line_number":181,"context_line":"            if not context.all_tenants:"},{"line_number":182,"context_line":"                # NOTE: The query doesn\u0027t work with table.c.tenant_id is None,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1e97c9fe_72496129","line":179,"in_reply_to":"20fa09bb_916ac4d9","updated":"2023-05-31 21:50:15.000000000","message":"Done","commit_id":"8255772a59b9501402b6e10427d09738f0431c2d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6185793748c8e60afee2b80030a1fd0a3c40124f","unresolved":true,"context_lines":[{"line_number":193,"context_line":"                                table.c.tenant_id \u003d\u003d None))  # NOQA"},{"line_number":194,"context_line":"                else:"},{"line_number":195,"context_line":"                    if include_shared:"},{"line_number":196,"context_line":"                        query \u003d query.where("},{"line_number":197,"context_line":"                            table.c.tenant_id \u003d\u003d context.project_id,"},{"line_number":198,"context_line":"                            _shared_zone_project_id \u003d\u003d context.project_id"},{"line_number":199,"context_line":"                        )"}],"source_content_type":"text/x-python","patch_set":5,"id":"375afe88_86ff0f4b","line":196,"updated":"2023-05-28 00:54:21.000000000","message":"Eric caught the missing _or here.","commit_id":"a31b985609653641828734b7429911ecedd83011"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f5df4d7a04b0fd9711005006f21e1f19e7f08d73","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                                table.c.tenant_id \u003d\u003d None))  # NOQA"},{"line_number":194,"context_line":"                else:"},{"line_number":195,"context_line":"                    if include_shared:"},{"line_number":196,"context_line":"                        query \u003d query.where("},{"line_number":197,"context_line":"                            table.c.tenant_id \u003d\u003d context.project_id,"},{"line_number":198,"context_line":"                            _shared_zone_project_id \u003d\u003d context.project_id"},{"line_number":199,"context_line":"                        )"}],"source_content_type":"text/x-python","patch_set":5,"id":"05b5627b_e0112b10","line":196,"in_reply_to":"375afe88_86ff0f4b","updated":"2023-05-28 00:55:28.000000000","message":"Done","commit_id":"a31b985609653641828734b7429911ecedd83011"}]}
