)]}'
{"nova/api/openstack/compute/used_limits.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4d54577900170064bc9e4ec5af5bcaf6a5bb96cb","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        used_limits \u003d {}"},{"line_number":62,"context_line":"        for display_name, key in quota_map.items():"},{"line_number":63,"context_line":"            if key in quotas:"},{"line_number":64,"context_line":"                reserved \u003d (quotas[key][\u0027reserved\u0027]"},{"line_number":65,"context_line":"                            if self._reserved(req) else 0)"},{"line_number":66,"context_line":"                used_limits[display_name] \u003d quotas[key][\u0027in_use\u0027] + reserved"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        resp_obj.obj[\u0027limits\u0027][\u0027absolute\u0027].update(used_limits)"}],"source_content_type":"text/x-python","patch_set":41,"id":"1f1a1f67_4cd8b1ed","side":"PARENT","line":65,"range":{"start_line":64,"start_character":16,"end_line":65,"end_character":58},"updated":"2017-07-19 02:19:54.000000000","message":"It wouldn\u0027t be a bad idea to add a TODO saying we should drop the reserved query parameter in a microversion. I was going to say we could say something in the API reference about it being vestigial, but that wouldn\u0027t be correct for older clouds that don\u0027t have this code yet.\n\nI\u0027ll at least make myself a TODO somewhere to work on a spec in Queens to remove the query parameter.","commit_id":"39fed19b12b539fdbb173a4ecd73d9a6998b285d"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"0960c8942884e0843657a3f9fed70f335803f526","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        used_limits \u003d {}"},{"line_number":62,"context_line":"        for display_name, key in quota_map.items():"},{"line_number":63,"context_line":"            if key in quotas:"},{"line_number":64,"context_line":"                reserved \u003d (quotas[key][\u0027reserved\u0027]"},{"line_number":65,"context_line":"                            if self._reserved(req) else 0)"},{"line_number":66,"context_line":"                used_limits[display_name] \u003d quotas[key][\u0027in_use\u0027] + reserved"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        resp_obj.obj[\u0027limits\u0027][\u0027absolute\u0027].update(used_limits)"}],"source_content_type":"text/x-python","patch_set":41,"id":"1f1a1f67_c4ec7ebc","side":"PARENT","line":65,"range":{"start_line":64,"start_character":16,"end_line":65,"end_character":58},"in_reply_to":"1f1a1f67_4cd8b1ed","updated":"2017-07-19 17:22:55.000000000","message":"So, the API docs mention it https://developer.openstack.org/api-ref/compute/?expanded\u003dshow-rate-and-absolute-limits-detail#show-rate-and-absolute-limits\n\nbut given it would require a new microversion for signaling its non-use, fine with the above.","commit_id":"39fed19b12b539fdbb173a4ecd73d9a6998b285d"}]}