)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":39015,"name":"Abhiram R N","display_name":"Abhiram Narayana","email":"arn@redhat.com","username":"abhiramnarayana"},"change_message_id":"2a8de6ccd4dcaa54fc015c5b1bf9be30d1c0d159","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"70806bc0_ca17a55b","updated":"2026-04-14 12:56:39.000000000","message":"hi Omer,\n\nI just had one comment in these changes. Kindly check it.","commit_id":"fd2a825d3898cd99988c15601b39c3108603681e"},{"author":{"_account_id":39015,"name":"Abhiram R N","display_name":"Abhiram Narayana","email":"arn@redhat.com","username":"abhiramnarayana"},"change_message_id":"9b4efbe97a4ece8abc5f6548a907e6ecb415bd2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e81c17b6_238da254","in_reply_to":"70806bc0_ca17a55b","updated":"2026-04-15 09:34:09.000000000","message":"Done","commit_id":"fd2a825d3898cd99988c15601b39c3108603681e"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"35d14f58966c1cf04e66130a6a72a9d44f726cea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"274c277a_d059a41b","updated":"2026-05-20 08:12:18.000000000","message":"I think we can merge it","commit_id":"6c62e5fca8e39cb3b8e41e11e1db813185b30990"},{"author":{"_account_id":39015,"name":"Abhiram R N","display_name":"Abhiram Narayana","email":"arn@redhat.com","username":"abhiramnarayana"},"change_message_id":"9b4efbe97a4ece8abc5f6548a907e6ecb415bd2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3ef693b0_23974d07","updated":"2026-04-15 09:34:09.000000000","message":"LGTM","commit_id":"6c62e5fca8e39cb3b8e41e11e1db813185b30990"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"b1c293399ba66072484cd73415069a11c02cb4a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b5a9fb20_16af390e","updated":"2026-05-18 12:50:47.000000000","message":"hello potential reviewers. I am planning to merge this patch soon","commit_id":"6c62e5fca8e39cb3b8e41e11e1db813185b30990"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"98e108d2111e741d4f4c22d56788f8e5bbca72e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4bd07276_d05dbfd5","updated":"2026-04-15 11:32:24.000000000","message":"recheck failure is not related to the patch","commit_id":"6c62e5fca8e39cb3b8e41e11e1db813185b30990"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"fed27199afa212add76ece65a0c027151d378f64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b2fa536b_485692e4","updated":"2026-04-15 10:11:31.000000000","message":"recheck failure is not related to the patch","commit_id":"6c62e5fca8e39cb3b8e41e11e1db813185b30990"}],"designate/storage/sqlalchemy/alembic/versions/d2a4aea32428_add_tsigkey_id_to_pool_nameservers.py":[{"author":{"_account_id":39015,"name":"Abhiram R N","display_name":"Abhiram Narayana","email":"arn@redhat.com","username":"abhiramnarayana"},"change_message_id":"2a8de6ccd4dcaa54fc015c5b1bf9be30d1c0d159","unresolved":true,"context_lines":[{"line_number":31,"context_line":"def upgrade() -\u003e None:"},{"line_number":32,"context_line":"    op.add_column("},{"line_number":33,"context_line":"        \u0027pool_nameservers\u0027,"},{"line_number":34,"context_line":"        sa.Column(\u0027tsigkey_id\u0027, sa.String(36), nullable\u003dTrue)"},{"line_number":35,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":2,"id":"3261ac86_ecb3433d","line":34,"updated":"2026-04-14 12:56:39.000000000","message":"In the tables.py file I see as below\n    Column(\u0027tsigkey_id\u0027, UUID(), nullable\u003dTrue),\n\nBut in this new file here it is using sa.String(36). Should it be also UUID() instead of sa.String(36) ?","commit_id":"fd2a825d3898cd99988c15601b39c3108603681e"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"32e61c32d7fe8ea705b3feb143c4d9d343a4fc2f","unresolved":true,"context_lines":[{"line_number":31,"context_line":"def upgrade() -\u003e None:"},{"line_number":32,"context_line":"    op.add_column("},{"line_number":33,"context_line":"        \u0027pool_nameservers\u0027,"},{"line_number":34,"context_line":"        sa.Column(\u0027tsigkey_id\u0027, sa.String(36), nullable\u003dTrue)"},{"line_number":35,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":2,"id":"345ab2c6_4ef31c94","line":34,"in_reply_to":"3261ac86_ecb3433d","updated":"2026-04-15 08:09:31.000000000","message":"It should be UUID, thanks, fixed.","commit_id":"fd2a825d3898cd99988c15601b39c3108603681e"},{"author":{"_account_id":39015,"name":"Abhiram R N","display_name":"Abhiram Narayana","email":"arn@redhat.com","username":"abhiramnarayana"},"change_message_id":"9b4efbe97a4ece8abc5f6548a907e6ecb415bd2c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"def upgrade() -\u003e None:"},{"line_number":32,"context_line":"    op.add_column("},{"line_number":33,"context_line":"        \u0027pool_nameservers\u0027,"},{"line_number":34,"context_line":"        sa.Column(\u0027tsigkey_id\u0027, sa.String(36), nullable\u003dTrue)"},{"line_number":35,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":2,"id":"c16c06ac_28483ccd","line":34,"in_reply_to":"345ab2c6_4ef31c94","updated":"2026-04-15 09:34:09.000000000","message":"Done","commit_id":"fd2a825d3898cd99988c15601b39c3108603681e"}]}