)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"9d20c28670d1e8a5435d52ff4413519098b724c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"69c4da40_d6094871","updated":"2022-07-29 23:09:53.000000000","message":"Very work-in-progress, just checkpointing","commit_id":"76ebe88c639ed1a7c294f60c9841446035826455"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"4b60b191cf7f73f5e604354669d6412cb8dcd658","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3e666bcc_00c431d9","updated":"2022-07-29 23:14:40.000000000","message":"Very work-in-progress, just checkpointing","commit_id":"0472270244cf4f03d3f90295d89c5240b7c61a0f"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"3b3cfa9cdd6198cbcc8c40f2cb6fe599d6e1b1df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e2ba28b2_f61330a2","updated":"2022-08-03 03:20:53.000000000","message":"check experimental","commit_id":"64e3ac9c205b0330c42392ff8b2c2c047f00d50b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"7476184399812cf2db2c93d863e7e1dff7f5ced2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0c5a9a37_ada1826c","updated":"2022-08-04 02:17:09.000000000","message":"We probably want a release note as well right?","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"243f59b9c036286ca1a1e2718cb024c46f4d0168","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"320d0699_f17ef3f0","updated":"2022-08-03 19:32:13.000000000","message":"check experimental","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"e2edde8e327f8550be9cc9bd42278d0aaef79bca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"dc3f1b92_208d4606","in_reply_to":"0c5a9a37_ada1826c","updated":"2022-08-04 17:09:55.000000000","message":"Yep! Good call, totally forgot. Especially true considering I am changing the requirements.","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"a9e3b8318851a73dfb2e660a7b788c16a6dafb22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"ac9dd5e9_3fdd8336","updated":"2022-08-05 16:13:16.000000000","message":"LGTM - great job","commit_id":"60c378763aa37246762ff210f024e2af019f6397"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"fbc7c880da45017df7184a44c671a9b47b8f340f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3061011a_3cd647b4","updated":"2022-08-29 21:16:05.000000000","message":"This has two reviews, including a core review, and has been open for a while. I\u0027m going to merge this to make sure it makes it into Zed.\nThe old migrate code is not maintained.","commit_id":"60c378763aa37246762ff210f024e2af019f6397"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"a0d49f0c023494ff46d413055c273ab8b246b67e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"5c0ce05a_2fa59e12","updated":"2022-08-30 02:22:36.000000000","message":"recheck DesignatePtrRecord","commit_id":"60c378763aa37246762ff210f024e2af019f6397"}],"designate/storage/impl_sqlalchemy/alembic/env.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"3b3cfa9cdd6198cbcc8c40f2cb6fe599d6e1b1df","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"#    a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":2,"id":"512a70f0_be11c6db","line":1,"updated":"2022-08-03 03:20:53.000000000","message":"Nit: different spacing from other files (noticed this on one of my PRs too)","commit_id":"0472270244cf4f03d3f90295d89c5240b7c61a0f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"e2edde8e327f8550be9cc9bd42278d0aaef79bca","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"#    a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":2,"id":"b27cc9fc_743faf2f","line":1,"in_reply_to":"512a70f0_be11c6db","updated":"2022-08-04 17:09:55.000000000","message":"Done","commit_id":"0472270244cf4f03d3f90295d89c5240b7c61a0f"}],"designate/tests/unit/storage/impl_sqlalchemy/alembic/test_legacy_utils.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"3227a71d0ad2082c6576cd15ce15230c92be24db","unresolved":true,"context_lines":[{"line_number":18,"context_line":"import designate.tests"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class TestLegacyUtils(designate.tests.TestCase):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @mock.patch(\u0027sqlalchemy.MetaData\u0027)"},{"line_number":24,"context_line":"    @mock.patch(\u0027alembic.op.get_bind\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"849b7acc_7b014634","line":21,"updated":"2022-08-04 02:25:51.000000000","message":"Nit: I prefer to use the barebones oslotest.base.BaseTestCase for all unit-tests, as the Designate TestCase usually adds 500ms overhead (due to having to instantiate a ton of unwanted code).","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"e2edde8e327f8550be9cc9bd42278d0aaef79bca","unresolved":true,"context_lines":[{"line_number":18,"context_line":"import designate.tests"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class TestLegacyUtils(designate.tests.TestCase):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @mock.patch(\u0027sqlalchemy.MetaData\u0027)"},{"line_number":24,"context_line":"    @mock.patch(\u0027alembic.op.get_bind\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"9af0acaf_65e1315f","line":21,"in_reply_to":"849b7acc_7b014634","updated":"2022-08-04 17:09:55.000000000","message":"Interesting, good to know. I switched this one, but the manage database needs the other framework as it expects some configuration information to be available and fails if I switch it.","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"b699b2c5a1e272254028cee441f28c51d8d4ced0","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import designate.tests"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class TestLegacyUtils(designate.tests.TestCase):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @mock.patch(\u0027sqlalchemy.MetaData\u0027)"},{"line_number":24,"context_line":"    @mock.patch(\u0027alembic.op.get_bind\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"4290b7cf_1fd77a30","line":21,"in_reply_to":"9af0acaf_65e1315f","updated":"2022-08-04 17:37:24.000000000","message":"Done","commit_id":"de0968a42a33603fcc905dc5aa9caf038b6c9d00"}]}