)]}'
{"keystoneclient/session.py":[{"author":{"_account_id":220,"name":"Haneef Ali","email":"haneef.ali@hp.com","username":"haneef"},"change_message_id":"fea89768d4a9e9de503f544598b5946affcde2b3","unresolved":false,"context_lines":[{"line_number":283,"context_line":""},{"line_number":284,"context_line":"        try:"},{"line_number":285,"context_line":"            resp \u003d self.session.request(method, url, **kwargs)"},{"line_number":286,"context_line":"        except requests.exceptions.SSLError:"},{"line_number":287,"context_line":"            msg \u003d \u0027SSL exception connecting to identity endpoint %s\u0027 % url"},{"line_number":288,"context_line":"            raise exceptions.SSLError(msg)"},{"line_number":289,"context_line":"        except requests.exceptions.Timeout:"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_dc3544a9","line":286,"updated":"2014-07-16 21:02:04.000000000","message":"I don\u0027t think this is correct. All the clients use sessions instead of different httpclients. e.g Cinder use the   keystoneclient sessions to get token and talk to cinder.  So if the connection to cinder fails, then we will be logging   \"SSL exception connecting to identity endpoint\" wheres the correct message would have been \"SSL exception connecting to cinder endpoint\"","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"},{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"cf57a4529608c84e8c838c5758bd9ee6fc7a0bd1","unresolved":false,"context_lines":[{"line_number":290,"context_line":"            msg \u003d \u0027Request to identity endpoint %s timed out\u0027 % url"},{"line_number":291,"context_line":"            raise exceptions.RequestTimeout(msg)"},{"line_number":292,"context_line":"        except requests.exceptions.ConnectionError:"},{"line_number":293,"context_line":"            msg \u003d \u0027Unable to establish connection to identity endpoint %s\u0027 % ("},{"line_number":294,"context_line":"                url)"},{"line_number":295,"context_line":"            raise exceptions.ConnectionRefused(msg)"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_5a14309f","line":293,"updated":"2014-06-13 22:12:49.000000000","message":"Remove parenthesis","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e03226c2d55a497c999c86916fe7bf8c16549b49","unresolved":false,"context_lines":[{"line_number":290,"context_line":"            msg \u003d \u0027Request to identity endpoint %s timed out\u0027 % url"},{"line_number":291,"context_line":"            raise exceptions.RequestTimeout(msg)"},{"line_number":292,"context_line":"        except requests.exceptions.ConnectionError:"},{"line_number":293,"context_line":"            msg \u003d \u0027Unable to establish connection to identity endpoint %s\u0027 % ("},{"line_number":294,"context_line":"                url)"},{"line_number":295,"context_line":"            raise exceptions.ConnectionRefused(msg)"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_847ea8e1","line":293,"in_reply_to":"1ae5cdf2_5a14309f","updated":"2014-06-26 19:18:36.000000000","message":"remove the parenthesis... in favor of what?","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"},{"author":{"_account_id":11333,"name":"David J Hu","email":"david.hu@hpe.com","username":"dhu"},"change_message_id":"b25d6bf2125943fa367ed72c70f67c2931231acf","unresolved":false,"context_lines":[{"line_number":290,"context_line":"            msg \u003d \u0027Request to identity endpoint %s timed out\u0027 % url"},{"line_number":291,"context_line":"            raise exceptions.RequestTimeout(msg)"},{"line_number":292,"context_line":"        except requests.exceptions.ConnectionError:"},{"line_number":293,"context_line":"            msg \u003d \u0027Unable to establish connection to identity endpoint %s\u0027 % ("},{"line_number":294,"context_line":"                url)"},{"line_number":295,"context_line":"            raise exceptions.ConnectionRefused(msg)"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_8ad29432","line":293,"in_reply_to":"1ae5cdf2_847ea8e1","updated":"2014-07-17 05:00:58.000000000","message":"Either way, the # of chars per line exceeded the limit.  \n\nWith or without( ) around url does not do anything to the msg.  Unless, it is a special Python trick that I am not familiar with.","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"},{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"d6a53f82f29c3c1cfbff998024f2b8d8982dd534","unresolved":false,"context_lines":[{"line_number":290,"context_line":"            msg \u003d \u0027Request to identity endpoint %s timed out\u0027 % url"},{"line_number":291,"context_line":"            raise exceptions.RequestTimeout(msg)"},{"line_number":292,"context_line":"        except requests.exceptions.ConnectionError:"},{"line_number":293,"context_line":"            msg \u003d \u0027Unable to establish connection to identity endpoint %s\u0027 % ("},{"line_number":294,"context_line":"                url)"},{"line_number":295,"context_line":"            raise exceptions.ConnectionRefused(msg)"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_8a5d34ef","line":293,"in_reply_to":"1ae5cdf2_847ea8e1","updated":"2014-07-17 04:55:40.000000000","message":"Sorry Dolph, I thought the parenthesis around url was not needed here.","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"},{"author":{"_account_id":11387,"name":"Sam Leong","email":"chio-fai-sam.leong@hpe.com","username":"Chioleong"},"change_message_id":"8ac8cff5fe17803ef88650950416801c4b53d299","unresolved":false,"context_lines":[{"line_number":290,"context_line":"            msg \u003d \u0027Request to identity endpoint %s timed out\u0027 % url"},{"line_number":291,"context_line":"            raise exceptions.RequestTimeout(msg)"},{"line_number":292,"context_line":"        except requests.exceptions.ConnectionError:"},{"line_number":293,"context_line":"            msg \u003d \u0027Unable to establish connection to identity endpoint %s\u0027 % ("},{"line_number":294,"context_line":"                url)"},{"line_number":295,"context_line":"            raise exceptions.ConnectionRefused(msg)"},{"line_number":296,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_00af432c","line":293,"in_reply_to":"baada198_8a5d34ef","updated":"2014-07-17 06:42:29.000000000","message":"I would say if parenthesis is used here, the other 2 above may also use to make it consistent.","commit_id":"318114c57517ebcbc2f2853620ce053c6324805c"}]}