)]}'
{"tempest/scenario/test_volume_migrate_attached.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c4094bedc54f067eca1a936140d05db3af556cd1","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":"        migration_policy \u003d \u0027on-demand\u0027"},{"line_number":85,"context_line":"        self.admin_volumes_client.retype_volume("},{"line_number":86,"context_line":"            volume_id, new_type\u003dnew_volume_type,"},{"line_number":87,"context_line":"            migration_policy\u003dmigration_policy)"},{"line_number":88,"context_line":"        waiters.wait_for_volume_retype(self.volumes_client,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_49271e14","line":85,"range":{"start_line":84,"start_character":0,"end_line":85,"end_character":48},"updated":"2018-01-01 04:29:50.000000000","message":"as Sean mentioned \u0027on-demand\u0027 migration policy need admin, we should mention that as notes here for not changing it in future. \n\nalso i am much interested to test retype with migration_policy as \u0027never\u0027 so that we can check the auth behavior in retype operation but that is for separate patch.","commit_id":"65e4f9b9d1f0c61636895cc133062bf0a6263848"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"56ae2d440642c29cc3dfc2f1e0a77918f05f5fff","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":"        migration_policy \u003d \u0027on-demand\u0027"},{"line_number":85,"context_line":"        self.admin_volumes_client.retype_volume("},{"line_number":86,"context_line":"            volume_id, new_type\u003dnew_volume_type,"},{"line_number":87,"context_line":"            migration_policy\u003dmigration_policy)"},{"line_number":88,"context_line":"        waiters.wait_for_volume_retype(self.volumes_client,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_d6d626c4","line":85,"range":{"start_line":84,"start_character":0,"end_line":85,"end_character":48},"in_reply_to":"9f91af0f_49271e14","updated":"2018-01-04 20:57:32.000000000","message":"Additional NOTE in this patch can make confusions because current patch is just a reverting patch, otherwise not.\nI have posted another patch which adds the NOTE https://review.openstack.org/531245","commit_id":"65e4f9b9d1f0c61636895cc133062bf0a6263848"}]}