)]}'
{"cinder/tests/unit/test_emc_vnxdirect.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"91f2c313ac380eaccdfb96d4532eb3b085ee722c","unresolved":false,"context_lines":[{"line_number":3993,"context_line":"            mock.call(*td.MIGRATION_VERIFY_CMD(6231), poll\u003dTrue),"},{"line_number":3994,"context_line":"            mock.call(*td.MIGRATION_VERIFY_CMD(6232), poll\u003dTrue),"},{"line_number":3995,"context_line":"            mock.call(*td.CREATE_CONSISTENCYGROUP_CMD("},{"line_number":3996,"context_line":"                      new_cg[\u0027id\u0027], [6231, 6232]), poll\u003dTrue),"},{"line_number":3997,"context_line":"            mock.call(*td.DELETE_CG_SNAPSHOT(copied_snap_name))]"},{"line_number":3998,"context_line":"        self.assertEqual(expect_cmd, fake_cli.call_args_list)"},{"line_number":3999,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_bb1a0e6d","line":3996,"updated":"2015-11-09 12:17:55.000000000","message":"These seem like worrying changes - it appears you\u0027ve changed what the test is actually testing for, without explanation, in a patch that is supposed to be py3 migration. Can we get some details on why this was changed, please?","commit_id":"f609367b151d64085db27609f5b4d6690cc9bbfc"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"d5079c47f4c2a58f6dbb1802d55a5c0957c52eb7","unresolved":false,"context_lines":[{"line_number":3993,"context_line":"            mock.call(*td.MIGRATION_VERIFY_CMD(6231), poll\u003dTrue),"},{"line_number":3994,"context_line":"            mock.call(*td.MIGRATION_VERIFY_CMD(6232), poll\u003dTrue),"},{"line_number":3995,"context_line":"            mock.call(*td.CREATE_CONSISTENCYGROUP_CMD("},{"line_number":3996,"context_line":"                      new_cg[\u0027id\u0027], [6231, 6232]), poll\u003dTrue),"},{"line_number":3997,"context_line":"            mock.call(*td.DELETE_CG_SNAPSHOT(copied_snap_name))]"},{"line_number":3998,"context_line":"        self.assertEqual(expect_cmd, fake_cli.call_args_list)"},{"line_number":3999,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_5be9a219","line":3996,"in_reply_to":"da85f559_bb1a0e6d","updated":"2015-11-09 12:20:10.000000000","message":"I had to update tests for the change explained in the commit message: \"\nCreateConsistencyGroupTask, WaitMigrationsCompleteTask: sort the lun\nidentifier keys to have a determinist order. (...)\".","commit_id":"f609367b151d64085db27609f5b4d6690cc9bbfc"}]}