)]}'
{"tempest/test.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0946c718fbddd10579c455951a15cbe6b93b173d","unresolved":false,"context_lines":[{"line_number":460,"context_line":"        except AttributeError:"},{"line_number":461,"context_line":"            domain \u003d \u0027Default\u0027"},{"line_number":462,"context_line":""},{"line_number":463,"context_line":"        if CONF.identity.auth_version \u003d\u003d \u0027v2\u0027:"},{"line_number":464,"context_line":"            return cred_client.get_creds_client(client, tenant_client,"},{"line_number":465,"context_line":"                                                roles_client,"},{"line_number":466,"context_line":"                                                project_domain_name\u003ddomain)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_1d2cad97","line":463,"updated":"2015-12-09 08:49:17.000000000","message":"I think in base patch we can have common logic as per comment which will avoid this another if condition.\n\nAs we keep adding the clients we use the same because at the end get_creds_client methods supposed to work for both v2 and v3","commit_id":"d498aefed64f4ef7e8d8305f873f9fc97d1c682a"}]}