)]}'
{"glanceclient/common/http.py":[{"author":{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},"change_message_id":"cb12fb3ddd865127b0d0adc25662d0325144c16b","unresolved":false,"context_lines":[{"line_number":323,"context_line":"        except ksc_exc.RequestTimeout as e:"},{"line_number":324,"context_line":"            conn_url \u003d self.get_endpoint(auth\u003dkwargs.get(\u0027auth\u0027))"},{"line_number":325,"context_line":"            conn_url \u003d \"%s/%s\" % (conn_url.rstrip(\u0027/\u0027), url.lstrip(\u0027/\u0027))"},{"line_number":326,"context_line":"            message \u003d (\"Error communicating with %(endpoint)s %(e)s\" %"},{"line_number":327,"context_line":"                       dict(url\u003dconn_url, e\u003de))"},{"line_number":328,"context_line":"            raise exc.InvalidEndpoint(message\u003dmessage)"},{"line_number":329,"context_line":"        except ksc_exc.ConnectionRefused as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_67db841c","line":326,"updated":"2015-07-06 23:48:55.000000000","message":"I\u0027m pretty sure it did not, because we also have to change \u0027%(endpoint)s\u0027 to \u0027%(url)s\u0027 . Now there will be KeyError: \u0027endpoint\u0027 if this exception occurs","commit_id":"67c88c0dcc8ac613f0110ddd7224ba8f6ddf160f"}],"glanceclient/tests/unit/test_http.py":[{"author":{"_account_id":7179,"name":"Brad Pokorny","email":"brad_pokorny@symantec.com","username":"bpokorny"},"change_message_id":"d5db9c74aadc7dc96eb16a542befe5c26e4f002f","unresolved":false,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    def test_connection_timeout(self):"},{"line_number":143,"context_line":"        \"\"\""},{"line_number":144,"context_line":"        should receive a InvalidEndpoint if connection timeout."},{"line_number":145,"context_line":"        \"\"\""},{"line_number":146,"context_line":"        def cb(request, context):"},{"line_number":147,"context_line":"            raise requests.exceptions.Timeout"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_f24f17bc","line":144,"updated":"2015-07-07 23:13:11.000000000","message":"This is a small thing, but if you need to submit another patch for any reason, suggest changing \"should\" -\u003e \"Should\" to stay consistent with other comments in this file and \"a InvalidEndpoint\" -\u003e \"an InvalidEndpoint\".","commit_id":"024d5fade22a82f54b8152de70b4d96dc05575db"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"61e718cbd1711e44ed269ef33dcf4cc8c3bf8f76","unresolved":false,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    def test_connection_timeout(self):"},{"line_number":143,"context_line":"        \"\"\""},{"line_number":144,"context_line":"        should receive a InvalidEndpoint if connection timeout."},{"line_number":145,"context_line":"        \"\"\""},{"line_number":146,"context_line":"        def cb(request, context):"},{"line_number":147,"context_line":"            raise requests.exceptions.Timeout"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_16fb8c7c","line":144,"in_reply_to":"ba3cc151_f24f17bc","updated":"2015-07-08 01:05:18.000000000","message":"Thanks,Brad.I\u0027ll submit a new one once Mike\u0027s patch be merged: \nhttps://review.openstack.org/#/c/199061/","commit_id":"024d5fade22a82f54b8152de70b4d96dc05575db"}]}