)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8086d4d06c0daf1e686a0f340d47501f17d0d6a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c9fe4faa_50dc3253","updated":"2024-08-29 04:39:46.000000000","message":"Looks OK to me. Still reviewing other patches in the series","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"1b0eda80d4d802ec59dab53f9c6078f2528ed02c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ec5d794_078de4bc","updated":"2024-08-26 20:20:10.000000000","message":"recheck","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"75f39604012894f6a9b6201b58ed1de4c2d17000","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4458102_9b419146","updated":"2024-08-22 22:10:33.000000000","message":"recheck","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"7c7064b19fbf9ce29aa78aff0511523738a285c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c96dd532_77e1a5d4","updated":"2024-08-25 10:48:53.000000000","message":"recheck","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"b325865cdc60ae48ebbac41736e977cc975a63fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fc5ed259_5b0be54b","updated":"2024-08-27 05:42:22.000000000","message":"recheck","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"a62da8d1fc9f42d9422281f1f6f0de3c8870a726","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f1951a93_2e41d761","updated":"2024-08-22 10:11:21.000000000","message":"recheck unrelated tempest failure","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2971090197765fe691f78fa87eeddf8308b166ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"be5afed3_962f555c","updated":"2024-08-22 13:52:36.000000000","message":"the shelve/unshelve test failure in the grenade job looks unrealted it think this is realted to a bug that i saw on a diffent patch set to its a latent issue","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"83b7659e72ca99040e1497a0e9ed4c15955bfc8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"853e4eca_19c76cc3","updated":"2024-08-30 09:16:17.000000000","message":"recheck","commit_id":"4e9ffc28d2ec1f2c6c055894c6a7e33e8505f692"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"9c0da15903cff7af8955a734464600c57b4612d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"df0e4578_41fc576b","updated":"2024-08-30 22:23:05.000000000","message":"recheck ceph tempest flakey","commit_id":"4e9ffc28d2ec1f2c6c055894c6a7e33e8505f692"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"88fd1ea0b638840b9b1ecc85395375e870a95ac4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6b0df26a_e47eddad","updated":"2024-11-29 14:28:51.000000000","message":"i do not believe the timeouts are related to your patch.\n\nim not entirely sure how i feel about the migration testing but over all i am still happy with this. so re-adding my +2","commit_id":"c7398a84333fd35e900e4775602ffbdbdce02740"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"ea2df17528bc4f64c9df81e9155cfc7b9694b7d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e2f9c1b0_05b7cf59","updated":"2024-12-01 08:25:57.000000000","message":"recheck","commit_id":"c7398a84333fd35e900e4775602ffbdbdce02740"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"900b29d5e9e81a24a8a6adde38f055314102c5ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9457d1a2_37dffae8","in_reply_to":"6b0df26a_e47eddad","updated":"2024-12-08 03:49:41.000000000","message":"Honestly, how would you improve the migration test? I\u0027m unsure what else to test there apart from that the new column exists?","commit_id":"c7398a84333fd35e900e4775602ffbdbdce02740"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ea679d2eb68d732f689f78f97faabfac3c822324","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9b565e4b_819a1cab","in_reply_to":"9457d1a2_37dffae8","updated":"2024-12-09 14:02:28.000000000","message":"in this change there isn\u0027t really much else you can do\nim just not sure how that can every fail if the migration applies.\nso it add value just limited.","commit_id":"c7398a84333fd35e900e4775602ffbdbdce02740"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"7c23cbb347e5c0fa01bf5119b39af7b2afdda398","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"20966cec_ba3b6e19","updated":"2024-12-31 03:12:27.000000000","message":"recheck","commit_id":"87f4fb8ab313d99d1674910e4537daf2848cb979"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"0becf31dc81d30c2045a4126656b10218719b1fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"c617535c_2306c540","updated":"2025-01-29 06:10:08.000000000","message":"recheck","commit_id":"f787c97e31ddcccf78b0ea48cb44837e62c23e82"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"42ea9a57dbc4e4d257490b372ecb8cf4f07c4f72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"bd154c4d_5edd0f25","updated":"2025-01-31 02:45:11.000000000","message":"recheck tempest timeout.","commit_id":"9b2753d216ef17a7d9d472728b82a7fc37cca014"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"5109ac40f8f620fe3b5eed6e23fa7aff9beb3c2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"005940f3_d0123403","updated":"2025-01-31 02:44:45.000000000","message":"retry tempest timeout","commit_id":"9b2753d216ef17a7d9d472728b82a7fc37cca014"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8df6c904b3b234bde75d1f2fa6c254177ca1b76b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"d7f82496_f66c2fc8","updated":"2025-02-05 03:13:11.000000000","message":"Looks OK to me. Will upgrade to +2 when the third patch is approved.","commit_id":"d5165c705bbcdf80887fb40a004c45359576b141"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"e2ec13775f41a27d73acfa4a532e5f735605bfaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"1dda7c96_f39b0b0e","updated":"2025-02-18 10:26:16.000000000","message":"Simple patch but again, will +W once we approve the whole branch","commit_id":"a3785ffe4a75839397094dd9437cc09a0920c8c8"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"a52c77cdcfaf372ac2e3f7c469b4a7b4b41217bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"b769075e_fb6997cd","updated":"2025-02-18 19:12:21.000000000","message":"recheck nova-multicell","commit_id":"aa57f49bcd0d521caab481d53125364d3a352f52"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8dc1d1a9e961888a89447c1a4db83fe32266720a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"b9c8b52b_ad9d4f15","updated":"2025-02-24 19:55:53.000000000","message":"recheck parent patch had a kernel panic","commit_id":"d8e95078cd51e5dae982c9f8651efb9813cba4f6"}],"nova/db/main/migrations/versions/2903cd72dc14_add_tls_port_to_console_auth_tokens.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"8086d4d06c0daf1e686a0f340d47501f17d0d6a2","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    with op.batch_alter_table(\u0027console_auth_tokens\u0027, schema\u003dNone) as batch_op:"},{"line_number":33,"context_line":"        batch_op.add_column(sa.Column(\u0027tls_port\u0027, sa.Integer()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"0b0f3034_7d70dffb","line":33,"range":{"start_line":33,"start_character":31,"end_line":33,"end_character":37},"updated":"2024-08-29 04:39:46.000000000","message":"Note to self: nullable defaults to True in this case.\nhttps://docs.sqlalchemy.org/en/20/core/metadata.html#sqlalchemy.schema.Column.params.nullable","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"5c29485f7999bf904df5207dcccf75a9cfe7ad8b","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def upgrade():"},{"line_number":32,"context_line":"    with op.batch_alter_table(\u0027console_auth_tokens\u0027, schema\u003dNone) as batch_op:"},{"line_number":33,"context_line":"        batch_op.add_column(sa.Column(\u0027tls_port\u0027, sa.Integer()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b673aa88_03aa8040","line":33,"range":{"start_line":33,"start_character":31,"end_line":33,"end_character":37},"in_reply_to":"0b0f3034_7d70dffb","updated":"2024-08-30 09:57:46.000000000","message":"Done","commit_id":"7145c1eba36175311d5efdad5fea6a26ca5f6a55"}]}
