)]}'
{"shade/__init__.py":[{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"e4262805b3d7fb1099a731fe434248b02d952e94","unresolved":false,"context_lines":[{"line_number":316,"context_line":"        if self._neutron_client is None:"},{"line_number":317,"context_line":"            endpoint \u003d self.get_endpoint("},{"line_number":318,"context_line":"                service_type\u003dself.get_service_type(\u0027network\u0027),"},{"line_number":319,"context_line":"                region_name\u003dself.region_name)"},{"line_number":320,"context_line":"            token \u003d self.auth_token"},{"line_number":321,"context_line":"            self._neutron_client \u003d neutron_client.Client("},{"line_number":322,"context_line":"                token\u003dtoken,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a930d6b_408e4e7a","line":319,"updated":"2015-01-26 16:54:10.000000000","message":"\u0027endpoint\u0027 appears unneeded now","commit_id":"7e6a552b3b9413ea2adf0353ce27899383b7f157"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"fb7c7c6aa71a7a4019927ac0b4035936b51eb00d","unresolved":false,"context_lines":[{"line_number":355,"context_line":"            endpoint \u003d self.keystone_session.get_endpoint("},{"line_number":356,"context_line":"                service_type\u003dservice_type,"},{"line_number":357,"context_line":"                interface\u003dself.endpoint_type,"},{"line_number":358,"context_line":"                region_name\u003dregion_name)"},{"line_number":359,"context_line":"        except Exception as e:"},{"line_number":360,"context_line":"            self.log.debug(\"keystone cannot get endpoint\", exc_info\u003dTrue)"},{"line_number":361,"context_line":"            raise OpenStackCloudException("}],"source_content_type":"text/x-python","patch_set":8,"id":"1a930d6b_5c446949","line":358,"updated":"2015-01-26 22:54:47.000000000","message":"Shouldn\u0027t we be using self.region_name here?  I\u0027m not sure why a parameter for region is being created. It\u0027s not used. If the plan is to use it, we should default it to self.region_name.\n\nIt turns out if it is actually None, HP cloud does the wrong thing and can return the wrong endpoint.","commit_id":"ab76e3a5afa17cafe966bbdbc5bad236b36d2c31"}]}