)]}'
{"tempest/services/identity/v3/json/identity_client.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"659e211d621bdf800dd0b126d921b6571be25bd0","unresolved":false,"context_lines":[{"line_number":456,"context_line":"    def __init__(self):"},{"line_number":457,"context_line":"        super(V3TokenClientJSON, self).__init__(None)"},{"line_number":458,"context_line":"        auth_url \u003d CONF.identity.uri_v3"},{"line_number":459,"context_line":"        if CONF.identity.auth_version is \u0027v3\u0027:"},{"line_number":460,"context_line":"            if not auth_url and CONF.identity_feature_enabled.api_v3:"},{"line_number":461,"context_line":"                raise exceptions.InvalidConfiguration(\u0027you must specify a v3 \u0027"},{"line_number":462,"context_line":"                                                      \u0027using if using the v3 \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7adec928_918a8e19","line":459,"updated":"2014-05-15 03:36:44.000000000","message":"So the V3TokenClient is used either for V3Token tests, which are executed if v3 is marked as available,0 or to get a token for any test is auth_version \u003d\u003d \"v3\".\nSo if auth_version is v3 and the uri_v3 is empty we should raise an exception.\nAnd if api_v3 is true and uri_v3 is empty we should raise an exception.\n\nSo the following should be enough:\n\nif not auth_url:\n   raise...\n\nIf we get to this point with api_v3 false and auth_version \u003d\u003d \u0027v2\u0027 something is very wrong in tempest code, and we should fail anyways.","commit_id":"7c868475514f36492880d1e6118347f6df0c917e"}],"tempest/services/identity/v3/xml/identity_client.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"659e211d621bdf800dd0b126d921b6571be25bd0","unresolved":false,"context_lines":[{"line_number":450,"context_line":"    def __init__(self):"},{"line_number":451,"context_line":"        super(V3TokenClientXML, self).__init__(None)"},{"line_number":452,"context_line":"        auth_url \u003d CONF.identity.uri_v3"},{"line_number":453,"context_line":"        if CONF.identity.auth_version is \u0027v3\u0027:"},{"line_number":454,"context_line":"            if not auth_url and CONF.identity_feature_enabled.api_v3:"},{"line_number":455,"context_line":"                raise exceptions.InvalidConfiguration(\u0027you must specify a v3 \u0027"},{"line_number":456,"context_line":"                                                      \u0027using if using the v3 \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7adec928_f1859207","line":453,"updated":"2014-05-15 03:36:44.000000000","message":"Comment from the json client applies here as well.","commit_id":"7c868475514f36492880d1e6118347f6df0c917e"}]}