)]}'
{"cinder/utils.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"b9223ea995f21fd1bca4784a21e8a4e1e5f33bd7","unresolved":false,"context_lines":[{"line_number":656,"context_line":"                                     min_length\u003dmin_length,"},{"line_number":657,"context_line":"                                     max_length\u003dmax_length)"},{"line_number":658,"context_line":"    except(ValueError, TypeError) as exc:"},{"line_number":659,"context_line":"        raise exception.InvalidInput(message\u003dexc.args[0])"},{"line_number":660,"context_line":""},{"line_number":661,"context_line":"_visible_admin_metadata_keys \u003d [\u0027readonly\u0027, \u0027attached_mode\u0027]"},{"line_number":662,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_bc983fc1","line":659,"updated":"2016-05-17 11:34:20.000000000","message":"-1: It would cleaner to do:\n\n raise exception.InvalidInput(reason\u003dexc)\n\nAnd it would also provide additional message \"Invalid input received\" making this consistent with other errors.\n\nAnother alternative would be to use six.text_type, because accessing the args array is not clean.","commit_id":"83f7ec3b7c5c44eb467f51afb9da4855622f11d6"}]}