)]}'
{"cinder/volume/drivers/datera/datera_common.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15077c58f87c23dce3eb1ce1f41d66a11328aaab","unresolved":false,"context_lines":[{"line_number":309,"context_line":"        msg \u003d _("},{"line_number":310,"context_line":"            \u0027Failed to make a request to Datera cluster endpoint due \u0027"},{"line_number":311,"context_line":"            \u0027to the following reason: %s\u0027) % six.text_type("},{"line_number":312,"context_line":"            str(ex))"},{"line_number":313,"context_line":"        LOG.error(msg)"},{"line_number":314,"context_line":"        raise exception.DateraAPIException(msg)"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_f3f1ac6c","line":312,"range":{"start_line":312,"start_character":12,"end_line":312,"end_character":15},"updated":"2017-10-25 09:09:40.000000000","message":"This is already being converted to a string by six.text_type correctly.","commit_id":"0857c72efc7281625ebc807971bf135643b4d733"}],"cinder/volume/drivers/dell_emc/vnx/client.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15077c58f87c23dce3eb1ce1f41d66a11328aaab","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        except storops_ex.VNXLunNotFoundError as ex:"},{"line_number":153,"context_line":"            LOG.info(\"LUN %(name)s is already deleted. This message can \""},{"line_number":154,"context_line":"                     \"be safely ignored. Message: %(msg)s\","},{"line_number":155,"context_line":"                     {\u0027name\u0027: name, \u0027msg\u0027: str(ex)})"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def cleanup_async_lun(self, name, force\u003dFalse):"},{"line_number":158,"context_line":"        \"\"\"Helper method to cleanup stuff for async migration."}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_73b37c99","line":155,"updated":"2017-10-25 09:09:40.000000000","message":"The storops exceptions do provide a message member, so no need to blindly change all of these.\n\nAnd str() should not be used. It\u0027s already being formatted into a string by the %s.","commit_id":"0857c72efc7281625ebc807971bf135643b4d733"}],"cinder/volume/drivers/dell_emc/vnx/utils.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15077c58f87c23dce3eb1ce1f41d66a11328aaab","unresolved":false,"context_lines":[{"line_number":138,"context_line":"                LOG.debug(\u0027Exception raised when executing %(condition_name)s\u0027"},{"line_number":139,"context_line":"                          \u0027in wait_until. Message: %(msg)s\u0027,"},{"line_number":140,"context_line":"                          {\u0027condition_name\u0027: condition.__name__,"},{"line_number":141,"context_line":"                           \u0027msg\u0027: str(ex)})"},{"line_number":142,"context_line":"        if test_value:"},{"line_number":143,"context_line":"            raise loopingcall.LoopingCallDone()"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_13a68854","line":141,"range":{"start_line":141,"start_character":34,"end_line":141,"end_character":37},"updated":"2017-10-25 09:09:40.000000000","message":"Get rid of str() call.","commit_id":"0857c72efc7281625ebc807971bf135643b4d733"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15077c58f87c23dce3eb1ce1f41d66a11328aaab","unresolved":false,"context_lines":[{"line_number":300,"context_line":"                raise exception.VolumeDriverException(message\u003dmsg)"},{"line_number":301,"context_line":"            return result_lun"},{"line_number":302,"context_line":"        except Exception as ex:"},{"line_number":303,"context_line":"            if (not multihostmap and hasattr(ex, \u0027message\u0027) and"},{"line_number":304,"context_line":"                    \u0027CMMVC6071E\u0027 in str(ex)):"},{"line_number":305,"context_line":"                LOG.error(\u0027storwize_svc_multihostmap_enabled is set \u0027"},{"line_number":306,"context_line":"                          \u0027to False, not allowing multi host mapping.\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_537520ca","line":303,"range":{"start_line":303,"start_character":37,"end_line":303,"end_character":59},"updated":"2017-10-25 09:09:40.000000000","message":"o_O","commit_id":"0857c72efc7281625ebc807971bf135643b4d733"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15077c58f87c23dce3eb1ce1f41d66a11328aaab","unresolved":false,"context_lines":[{"line_number":1330,"context_line":"                              \u0027_wait_for_condition: %(method_name)s \u0027"},{"line_number":1331,"context_line":"                              \u0027execution failed for %(exception)s.\u0027,"},{"line_number":1332,"context_line":"                              {\u0027method_name\u0027: testmethod.__name__,"},{"line_number":1333,"context_line":"                               \u0027exception\u0027: str(ex)})"},{"line_number":1334,"context_line":"            if testValue:"},{"line_number":1335,"context_line":"                raise loopingcall.LoopingCallDone()"},{"line_number":1336,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_d352303b","line":1333,"range":{"start_line":1333,"start_character":44,"end_line":1333,"end_character":47},"updated":"2017-10-25 09:09:40.000000000","message":"Do not convert to string ahead of it being converted to a string.","commit_id":"0857c72efc7281625ebc807971bf135643b4d733"}]}