)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a727b3950979d7689a6241d90edbfa980c4d0233","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-07-28 20:29:58 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add regression test for bug 1938326"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1938326"},{"line_number":10,"context_line":"Change-Id: Ie7fa120cb7d1739c5de7cd8dbcfaeb3a37fa729b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bd13103e_61936fa0","line":8,"updated":"2021-07-29 11:09:54.000000000","message":"A quick summary of the bug here would be helpful IMO","commit_id":"b7a8818bd01dbd66ee8a7374b4e712581f513faa"}],"nova/tests/functional/regressions/test_bug_1938326.py":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"84b034f5b5d29f486ec4a154ccd7d267e1e5317b","unresolved":true,"context_lines":[{"line_number":140,"context_line":"            # We don\u0027t want to wait here so just make the raw api call"},{"line_number":141,"context_line":"            self.api.post_server_action(server[\u0027id\u0027], {\u0027migrate\u0027: None})"},{"line_number":142,"context_line":"            # Assert that the dest compute attemptes to call resize_instance"},{"line_number":143,"context_line":"            mock_dest_rpcapi_resize.assert_called_once()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5ec4455a_e0d7be3f","line":143,"updated":"2021-07-30 09:57:15.000000000","message":"I\u0027m being a bit nitpicky here, but could we not assert that the instance remains in pre-migrating - or I guess more specifically, that it\u0027s in pre-migrating, and that we timeout waiting for it to leave that state and transition into the next state?","commit_id":"b7a8818bd01dbd66ee8a7374b4e712581f513faa"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"0a1bc6c28fcc19a42c8e4850d153cc99633139ce","unresolved":true,"context_lines":[{"line_number":140,"context_line":"            # We don\u0027t want to wait here so just make the raw api call"},{"line_number":141,"context_line":"            self.api.post_server_action(server[\u0027id\u0027], {\u0027migrate\u0027: None})"},{"line_number":142,"context_line":"            # Assert that the dest compute attemptes to call resize_instance"},{"line_number":143,"context_line":"            mock_dest_rpcapi_resize.assert_called_once()"}],"source_content_type":"text/x-python","patch_set":1,"id":"4a3343db_98bc42f5","line":143,"in_reply_to":"5ec4455a_e0d7be3f","updated":"2021-08-02 10:26:49.000000000","message":"Possibly, I\u0027m fine with asserting that we get to pre-migrating but waiting for it to timeout getting to migrating feels wrong in a functional test when we can just assert the RPC interface on the dest.","commit_id":"b7a8818bd01dbd66ee8a7374b4e712581f513faa"}]}