While trying to remove the skips from the identity tests I discovered this general bug. The particular case I found was this code:
if resp.status == 400:
resp_body = json.loads(resp_body)
self._log(req_url, body, resp, resp_body)
raise exceptions.BadRequest(resp_body['badRequest']['message'])
This assumes that the response body is nested json tagged by "badRequest" and then "message" which is what nova does. It seems that 400 errors from keystone give ["error"]["message"]. We could abstract this behavior into a RestClient method that could be overridden by specific client, but I don't think we should be relying on this at all. It is not part of the spec as far as I can see. I think it would be better to just put the raw json string as the message "body". Comments?
Yeah, this is true. I fixed this up for the 500 error series (looking up various things like 'computeFault', 'cloudServersFa
You can go either route I suppose... or do something like check for 'badRequest in body, and if so, grab the message, if not just dump the body text/json to the exception raised.