)]}'
{"tempest/scenario/test_volume_migrate_attached.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"05f9ca43bfb01e556fa0e275846940a4653ed9ef","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        return source_body[\u0027name\u0027], dest_body[\u0027name\u0027]"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    def _volume_retype_with_migration(self, volume_id, new_volume_type):"},{"line_number":84,"context_line":"        # NOTE: The \u0027on-demand\u0027 migration requires admin operation, so we"},{"line_number":85,"context_line":"        # should not remove admin_volumes_client() usage here."},{"line_number":86,"context_line":"        migration_policy \u003d \u0027on-demand\u0027"},{"line_number":87,"context_line":"        self.admin_volumes_client.retype_volume("},{"line_number":88,"context_line":"            volume_id, new_type\u003dnew_volume_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_5208d92a","line":85,"range":{"start_line":84,"start_character":68,"end_line":85,"end_character":61},"updated":"2018-01-05 08:09:34.000000000","message":"How about \"so admin_volumes_client() should be used here\"?\nBecause it\u0027s a bit strange to mention \"should not remove\" :)","commit_id":"1095c6d3733fb317507dc2cfaddf45addc3b7f12"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0f8133e0b029e37112127a2dd6acafb9cffe435f","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        return source_body[\u0027name\u0027], dest_body[\u0027name\u0027]"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    def _volume_retype_with_migration(self, volume_id, new_volume_type):"},{"line_number":84,"context_line":"        # NOTE: The \u0027on-demand\u0027 migration requires admin operation, so we"},{"line_number":85,"context_line":"        # should not remove admin_volumes_client() usage here."},{"line_number":86,"context_line":"        migration_policy \u003d \u0027on-demand\u0027"},{"line_number":87,"context_line":"        self.admin_volumes_client.retype_volume("},{"line_number":88,"context_line":"            volume_id, new_type\u003dnew_volume_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_cfc40e94","line":85,"range":{"start_line":84,"start_character":68,"end_line":85,"end_character":61},"in_reply_to":"9f91af0f_5208d92a","updated":"2018-01-08 18:18:04.000000000","message":"hehe, right.","commit_id":"1095c6d3733fb317507dc2cfaddf45addc3b7f12"}]}