)]}'
{"cinderclient/client.py":[{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"1c10cb7f6cf9183fbcc96af152a604f813c6705a","unresolved":false,"context_lines":[{"line_number":372,"context_line":"    def get_volume_api_version_from_endpoint(self):"},{"line_number":373,"context_line":"        magic_tuple \u003d urlparse.urlsplit(self.management_url)"},{"line_number":374,"context_line":"        scheme, netloc, path, query, frag \u003d magic_tuple"},{"line_number":375,"context_line":"        return path.lstrip(\u0027/\u0027).split(\u0027/\u0027)[0][1:]"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"def get_client_class(version):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2FwJ8%3D","line":375,"updated":"2013-06-11 12:04:30.000000000","message":"you may not need lstrip() here, just split(\u0027/\u0027) and use second element\nconsider doing some error checking on result of this operation in case the management_url doesn\u0027t match expected format","commit_id":"1605c5c19343eaf0575aa296343d69381c6ef2da"}],"cinderclient/exceptions.py":[{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"1c10cb7f6cf9183fbcc96af152a604f813c6705a","unresolved":false,"context_lines":[{"line_number":13,"context_line":"class InvalidAPIVersion(Exception):"},{"line_number":14,"context_line":"    \"\"\"Indicates that the user is trying to use API version"},{"line_number":15,"context_line":"    which is different from the endpoint version.\"\"\""},{"line_number":16,"context_line":"    def __init__(self, message\u003dNone):"},{"line_number":17,"context_line":"        self.message \u003d message"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2FwJk%3D","line":16,"updated":"2013-06-11 12:04:30.000000000","message":"I think this code is unneccesary","commit_id":"1605c5c19343eaf0575aa296343d69381c6ef2da"}],"cinderclient/shell.py":[{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"1c10cb7f6cf9183fbcc96af152a604f813c6705a","unresolved":false,"context_lines":[{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        endpoint_api_version \u003d self.cs.get_volume_api_version_from_endpoint()"},{"line_number":452,"context_line":"        if endpoint_api_version !\u003d options.os_volume_api_version:"},{"line_number":453,"context_line":"            msg \u003d ((\"env[OS_VOLUME_API_VERSION] is set to %s \""},{"line_number":454,"context_line":"                    \"but you are accessing a %s endpoint\")"},{"line_number":455,"context_line":"                   % (options.os_volume_api_version, endpoint_api_version))"},{"line_number":456,"context_line":"            raise exc.InvalidAPIVersion(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2FwJc%3D","line":453,"updated":"2013-06-11 12:04:30.000000000","message":"Thinking about the error message, you can probably trigger it with something like \u0027cinder --os-volume-api-version N ...\u0027 also so maybe make the error message more generic to apply to that also","commit_id":"1605c5c19343eaf0575aa296343d69381c6ef2da"}]}