)]}'
{"saharaclient/api/client.py":[{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"89fd1e756a3622b37d30bc75457673ef02f538fb","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            raise RuntimeError(\"Could not find Sahara endpoint in catalog\")"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        self.client \u003d httpclient.HTTPClient(sahara_catalog_url,"},{"line_number":66,"context_line":"                                            input_auth_token)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        self.clusters \u003d clusters.ClusterManager(self)"},{"line_number":69,"context_line":"        self.cluster_templates \u003d cluster_templates.ClusterTemplateManager(self)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2F9M4%3D","line":66,"updated":"2014-03-12 17:46:16.000000000","message":"do we have pep8 enabled for client? Don\u0027t think this is right formatting. But it was here before.","commit_id":"2feb1aed61dcb0de8fd0ae12b8bbe5a8f97bc4cd"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"ca3d5883f73b03c2ef11e16c5b1b4c4de0576355","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            raise RuntimeError(\"Could not find Sahara endpoint in catalog\")"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        self.client \u003d httpclient.HTTPClient(sahara_catalog_url,"},{"line_number":66,"context_line":"                                            input_auth_token)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        self.clusters \u003d clusters.ClusterManager(self)"},{"line_number":69,"context_line":"        self.cluster_templates \u003d cluster_templates.ClusterTemplateManager(self)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2F8%2F4%3D","line":66,"in_reply_to":"AAAAWH%2F%2F9M4%3D","updated":"2014-03-12 18:01:42.000000000","message":"I think so.  I just added a bunch of spaces before a class declaration and ran tox -e pep8, it picked up the extra spaces.  Maybe there is a rule that needs to be added?  What do you think is incorrect?","commit_id":"2feb1aed61dcb0de8fd0ae12b8bbe5a8f97bc4cd"},{"author":{"_account_id":7555,"name":"Matthew Farrellee","email":"matt@redhat.com","username":"farrellee"},"change_message_id":"9b2e068a0b9a5c1f0c3989f94683c3bb83aea4d7","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class Client(object):"},{"line_number":33,"context_line":"    def __init__(self, username\u003dNone, api_key\u003dNone, project_id\u003dNone,"},{"line_number":34,"context_line":"                 project_name\u003dNone, auth_url\u003dNone, sahara_url\u003dNone,"},{"line_number":35,"context_line":"                 endpoint_type\u003d\u0027publicURL\u0027, service_type\u003d\u0027data_processing\u0027,"},{"line_number":36,"context_line":"                 input_auth_token\u003dNone):"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F7iM%3D","line":34,"updated":"2014-03-12 21:17:48.000000000","message":"does the alias client map savanna_url to sahara_url?","commit_id":"77532a6a8ba561a6138fde5c306b08ec5ca93cc6"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"31b07118269558921445d3fb47cb5fc64888e837","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class Client(object):"},{"line_number":33,"context_line":"    def __init__(self, username\u003dNone, api_key\u003dNone, project_id\u003dNone,"},{"line_number":34,"context_line":"                 project_name\u003dNone, auth_url\u003dNone, sahara_url\u003dNone,"},{"line_number":35,"context_line":"                 endpoint_type\u003d\u0027publicURL\u0027, service_type\u003d\u0027data_processing\u0027,"},{"line_number":36,"context_line":"                 input_auth_token\u003dNone):"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F1BY%3D","line":34,"in_reply_to":"AAAAWH%2F%2F10o%3D","updated":"2014-03-13 15:50:47.000000000","message":"Done, added to savannaclient.client in this CR","commit_id":"77532a6a8ba561a6138fde5c306b08ec5ca93cc6"},{"author":{"_account_id":7555,"name":"Matthew Farrellee","email":"matt@redhat.com","username":"farrellee"},"change_message_id":"22f76364ae5762104aeaa975a91b80d761303c0b","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class Client(object):"},{"line_number":33,"context_line":"    def __init__(self, username\u003dNone, api_key\u003dNone, project_id\u003dNone,"},{"line_number":34,"context_line":"                 project_name\u003dNone, auth_url\u003dNone, sahara_url\u003dNone,"},{"line_number":35,"context_line":"                 endpoint_type\u003d\u0027publicURL\u0027, service_type\u003d\u0027data_processing\u0027,"},{"line_number":36,"context_line":"                 input_auth_token\u003dNone):"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F10o%3D","line":34,"in_reply_to":"AAAAWH%2F%2F7OI%3D","updated":"2014-03-13 13:10:18.000000000","message":"you\u0027re right, this should be addressed at the layer that handles backward compat, not here.","commit_id":"77532a6a8ba561a6138fde5c306b08ec5ca93cc6"},{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"f21bd1a59a49b2ecd17d2491774236f0aeba751e","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class Client(object):"},{"line_number":33,"context_line":"    def __init__(self, username\u003dNone, api_key\u003dNone, project_id\u003dNone,"},{"line_number":34,"context_line":"                 project_name\u003dNone, auth_url\u003dNone, sahara_url\u003dNone,"},{"line_number":35,"context_line":"                 endpoint_type\u003d\u0027publicURL\u0027, service_type\u003d\u0027data_processing\u0027,"},{"line_number":36,"context_line":"                 input_auth_token\u003dNone):"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F7OI%3D","line":34,"in_reply_to":"AAAAWH%2F%2F7iM%3D","updated":"2014-03-12 21:55:25.000000000","message":"See possible solution in https://review.openstack.org/80096\n\nThe other way to handle this is add \"savanna_url\u003dNone\" to the end of the parameter list.","commit_id":"77532a6a8ba561a6138fde5c306b08ec5ca93cc6"}]}