)]}'
{"horizon/api/nova.py":[{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"a3daadf88b8e5b94ec9644f38dd93d9414e3e855","unresolved":false,"context_lines":[{"line_number":399,"context_line":""},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"def tenant_quota_get(request, tenant_id):"},{"line_number":402,"context_line":"    return QuotaSet(novaclient(request).quotas.get(tenant_id))"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"def tenant_quota_update(request, tenant_id, **kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FtS4%3D","side":"PARENT","line":402,"updated":"2013-01-09 02:19:17.000000000","message":"This code originally grabbed quotas from nova, and this is changing it to use cinder? I think you can leave it as it was (support for retrieving volume quotas directly from cinder was added post-Folsom).","commit_id":"1d1fa854dd54b9b27b7eb67e195c0a7cdeaeba08"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"9b77f825eaa6b765bfab30029186c9bcad69f9c9","unresolved":false,"context_lines":[{"line_number":399,"context_line":""},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"def tenant_quota_get(request, tenant_id):"},{"line_number":402,"context_line":"    return QuotaSet(novaclient(request).quotas.get(tenant_id))"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"def tenant_quota_update(request, tenant_id, **kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FsxQ%3D","side":"PARENT","line":402,"in_reply_to":"AAAAM3%2F%2FtS4%3D","updated":"2013-01-09 07:34:23.000000000","message":"Ah yes, good catch! Will change that back.","commit_id":"1d1fa854dd54b9b27b7eb67e195c0a7cdeaeba08"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"b094cb0399c7b1a24f1b797a567cabdb56bd1b1e","unresolved":false,"context_lines":[{"line_number":428,"context_line":"    floating_ips \u003d tenant_floating_ip_list(request)"},{"line_number":429,"context_line":"    quotas \u003d tenant_quota_get(request, request.user.tenant_id)"},{"line_number":430,"context_line":"    flavors \u003d dict([(f.id, f) for f in flavor_list(request)])"},{"line_number":431,"context_line":"    volumes \u003d volume_list(request)"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    usages \u003d {\u0027instances\u0027: {\u0027flavor_fields\u0027: [], \u0027used\u0027: len(instances)},"},{"line_number":434,"context_line":"              \u0027cores\u0027: {\u0027flavor_fields\u0027: [\u0027vcpus\u0027], \u0027used\u0027: 0},"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FqvI%3D","side":"PARENT","line":431,"updated":"2013-01-10 06:34:57.000000000","message":"This fails as well. volume_list() needs the same treatment as volume_snapshot_list().","commit_id":"1d1fa854dd54b9b27b7eb67e195c0a7cdeaeba08"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"7e6273d51c1e69ef2c26189a1cd6afe599d5c48a","unresolved":false,"context_lines":[{"line_number":428,"context_line":"    floating_ips \u003d tenant_floating_ip_list(request)"},{"line_number":429,"context_line":"    quotas \u003d tenant_quota_get(request, request.user.tenant_id)"},{"line_number":430,"context_line":"    flavors \u003d dict([(f.id, f) for f in flavor_list(request)])"},{"line_number":431,"context_line":"    volumes \u003d volume_list(request)"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    usages \u003d {\u0027instances\u0027: {\u0027flavor_fields\u0027: [], \u0027used\u0027: len(instances)},"},{"line_number":434,"context_line":"              \u0027cores\u0027: {\u0027flavor_fields\u0027: [\u0027vcpus\u0027], \u0027used\u0027: 0},"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FqlQ%3D","side":"PARENT","line":431,"in_reply_to":"AAAAM3%2F%2FqvI%3D","updated":"2013-01-10 10:54:40.000000000","message":"right, that one is missing in master, too. Will submit it there also.","commit_id":"1d1fa854dd54b9b27b7eb67e195c0a7cdeaeba08"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"b094cb0399c7b1a24f1b797a567cabdb56bd1b1e","unresolved":false,"context_lines":[{"line_number":216,"context_line":"    except exceptions.ServiceCatalogException:"},{"line_number":217,"context_line":"        LOG.debug("},{"line_number":218,"context_line":"            \u0027cinderclient connection created using token \"%s\" and url \"%s\"\u0027 %"},{"line_number":219,"context_line":"            (request.user.token.id, cinder_url))"},{"line_number":220,"context_line":"    c \u003d cinder_client.Client(request.user.username,"},{"line_number":221,"context_line":"                             request.user.token.id,"},{"line_number":222,"context_line":"                             project_id\u003drequest.user.tenant_id,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FqvY%3D","line":219,"updated":"2013-01-10 06:34:57.000000000","message":"return None here?\n\nAnd the message doesn\u0027t make sense. I\u0027m guessing git made a mess of the merge? Refer to the original patch.","commit_id":"beeba4174c1546cfae782790e71e93ff18c839b6"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"7e6273d51c1e69ef2c26189a1cd6afe599d5c48a","unresolved":false,"context_lines":[{"line_number":216,"context_line":"    except exceptions.ServiceCatalogException:"},{"line_number":217,"context_line":"        LOG.debug("},{"line_number":218,"context_line":"            \u0027cinderclient connection created using token \"%s\" and url \"%s\"\u0027 %"},{"line_number":219,"context_line":"            (request.user.token.id, cinder_url))"},{"line_number":220,"context_line":"    c \u003d cinder_client.Client(request.user.username,"},{"line_number":221,"context_line":"                             request.user.token.id,"},{"line_number":222,"context_line":"                             project_id\u003drequest.user.tenant_id,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FqlU%3D","line":219,"in_reply_to":"AAAAM3%2F%2FqvY%3D","updated":"2013-01-10 10:54:40.000000000","message":"You\u0027re right. Will double check the full commit.","commit_id":"beeba4174c1546cfae782790e71e93ff18c839b6"}]}