)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0f9992f5152564a929704c7e12e37599ef0f2873","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  in nova/tests/unit/compute/test_compute.py"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The test method has been changed since"},{"line_number":16,"context_line":"I6d422e0e6d29492a1c449039c04f6d38afa94d5c."},{"line_number":17,"context_line":"It needed a follow-up patch I82a8619229164c10f9b38804cee86de0876e4df8,"},{"line_number":18,"context_line":"but the patch has not been merged."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_9e51d165","line":16,"updated":"2019-07-23 18:41:48.000000000","message":"Even before this the test never really seemed to do anything useful correct? It was just calling pre_live_migration on the fake virt driver used in the test.\n\nIt looks like I2c4d4dae2cb0f00c67f0f731b5080196564b37da is what broke the test and really the test is not very useful anymore - why don\u0027t we just delete it?","commit_id":"794293f056a1e6d5bcad0758ddb81727b9a03863"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"30008f397e903c2e185673cf252d08e30272f90c","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  in nova/tests/unit/compute/test_compute.py"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The test method has been changed since"},{"line_number":16,"context_line":"I6d422e0e6d29492a1c449039c04f6d38afa94d5c."},{"line_number":17,"context_line":"It needed a follow-up patch I82a8619229164c10f9b38804cee86de0876e4df8,"},{"line_number":18,"context_line":"but the patch has not been merged."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_676fa192","line":16,"in_reply_to":"7faddb67_9e51d165","updated":"2019-07-24 01:43:46.000000000","message":"Done","commit_id":"794293f056a1e6d5bcad0758ddb81727b9a03863"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"22f13283442f632fe117659e2d50cd99b46fea50","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove test_pre_live_migration_instance_has_no_fixed_ip"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove the following unit test"},{"line_number":10,"context_line":"because it is not useful any more."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"* test_pre_live_migration_instance_has_no_fixed_ip"},{"line_number":13,"context_line":"  in nova/tests/unit/compute/test_compute.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7faddb67_01b1f2e1","line":10,"updated":"2019-07-24 13:29:36.000000000","message":"Would have been good to explain in the commit message why it\u0027s not useful anymore, but essentially because of I2c4d4dae2cb0f00c67f0f731b5080196564b37da and how that change broke the test to call the driver rather than the compute manager code.","commit_id":"cdb752f3d3601cb8cca48f28d6b95d2dbc9e1e20"}]}