)]}'
{"cinder/volume/drivers/dell_emc/vmax/common.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9d3bec51b280e29511e1707356291ada19588227","unresolved":false,"context_lines":[{"line_number":4392,"context_line":"                \"Volume is replicated - revert to snapshot feature is not \""},{"line_number":4393,"context_line":"                \"supported for replicated volumes.\"))"},{"line_number":4394,"context_line":"            LOG.error(exception_message)"},{"line_number":4395,"context_line":"            raise exception.VolumeDriverException(data\u003dexception_message)"},{"line_number":4396,"context_line":"        array \u003d extra_specs[utils.ARRAY]"},{"line_number":4397,"context_line":"        sourcedevice_id, snap_name \u003d self._parse_snap_info("},{"line_number":4398,"context_line":"            array, snapshot)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_27bbd50f","line":4395,"range":{"start_line":4395,"start_character":28,"end_line":4395,"end_character":73},"updated":"2018-06-29 03:14:08.000000000","message":"You should use \u0027\u0027exception.VolumeDriverException(message\u003dexception_message)\u0027\u0027 or exception.VolumeDriverException(exception_message) instead, otherwise this exception detail can\u0027t be recognized.","commit_id":"e68d2f7a932107e711abffc797ef7aab733d7a67"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"9112cbfa0d9cd5979a695f807033e27018e166f6","unresolved":false,"context_lines":[{"line_number":4392,"context_line":"                \"Volume is replicated - revert to snapshot feature is not \""},{"line_number":4393,"context_line":"                \"supported for replicated volumes.\"))"},{"line_number":4394,"context_line":"            LOG.error(exception_message)"},{"line_number":4395,"context_line":"            raise exception.VolumeDriverException(data\u003dexception_message)"},{"line_number":4396,"context_line":"        array \u003d extra_specs[utils.ARRAY]"},{"line_number":4397,"context_line":"        sourcedevice_id, snap_name \u003d self._parse_snap_info("},{"line_number":4398,"context_line":"            array, snapshot)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_3bec4d91","line":4395,"range":{"start_line":4395,"start_character":28,"end_line":4395,"end_character":73},"in_reply_to":"5f7c97a3_27bbd50f","updated":"2018-06-29 09:06:08.000000000","message":"Thanks Tommy, We will need to change it in other places also.  I will open a separate review for that.","commit_id":"e68d2f7a932107e711abffc797ef7aab733d7a67"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9d3bec51b280e29511e1707356291ada19588227","unresolved":false,"context_lines":[{"line_number":4400,"context_line":"            LOG.error(\"No snapshot found on the array\")"},{"line_number":4401,"context_line":"            exception_message \u003d (_("},{"line_number":4402,"context_line":"                \"Failed to revert the volume to the snapshot\"))"},{"line_number":4403,"context_line":"            raise exception.VolumeDriverException(data\u003dexception_message)"},{"line_number":4404,"context_line":"        self._sync_check(array, sourcedevice_id, volume.name, extra_specs)"},{"line_number":4405,"context_line":"        try:"},{"line_number":4406,"context_line":"            LOG.info(\"Reverting device: %(deviceid)s \""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_c741c10c","line":4403,"range":{"start_line":4403,"start_character":50,"end_line":4403,"end_character":56},"updated":"2018-06-29 03:14:08.000000000","message":"ditto.","commit_id":"e68d2f7a932107e711abffc797ef7aab733d7a67"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"14bfc3ac43e1b4da520941ac04ae506c788b5a8f","unresolved":false,"context_lines":[{"line_number":4387,"context_line":"        \"\"\""},{"line_number":4388,"context_line":"        extra_specs \u003d self._initial_setup(volume)"},{"line_number":4389,"context_line":"        if self.utils.is_replication_enabled(extra_specs):"},{"line_number":4390,"context_line":"            exception_message \u003d (_("},{"line_number":4391,"context_line":"                \"Volume is replicated - revert to snapshot feature is not \""},{"line_number":4392,"context_line":"                \"supported for replicated volumes.\"))"},{"line_number":4393,"context_line":"            LOG.error(exception_message)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5f7c97a3_33c91f50","line":4390,"range":{"start_line":4390,"start_character":32,"end_line":4390,"end_character":33},"updated":"2018-07-20 12:53:10.000000000","message":"Outer () not needed.","commit_id":"92fbe37087f75ac698e39f2fd635caefac1a4e6d"}]}