Cinder extension to add used resources in absolute limits

Bug #1240789 reported by OpenStack Infra
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-api-site
Fix Released
Medium
Diane Fleming

Bug Description

https://review.openstack.org/49642
commit 8aa8ebc0a0d0f459e9293f9ff5a5f8358fd01b9a
Author: Olivier Pilotte <email address hidden>
Date: Thu Oct 3 18:10:40 2013 -0400

    Cinder extension to add used resources in absolute limits

    * totalVolumesUsed
    * totalGigabytesUsed

    DocImpact
    Implements: blueprint cinder-used-resources-in-absolute-limits
    Change-Id: I525a61043ce2f906c424738ab361ee7396f072b5

Tags: volume-api
Tom Fifield (fifieldt)
affects: openstack-manuals → openstack-api-site
Changed in openstack-api-site:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → icehouse
Changed in openstack-api-site:
assignee: nobody → Miranda Zhang (miranda-zhang-q)
Revision history for this message
Miranda Zhang (miranda-zhang-q) wrote :

https://review.openstack.org/49642
this patch doesn't include any example output

it looks like the modification affects this api call:
http://api.openstack.org/api-ref-compute.html#limits

take the am I suppose to add

"totalVolumesUsed": 0,
"totalGigabytesUsed": 0

to the json response:
https://github.com/openstack/api-site/blob/master/api-ref/src/wadls/compute-api/src/api_samples/os-used-limits-for-admin/usedlimitsforadmin-get-resp.json

and add

    <limit name="totalVolumesUsed" value="0"/>
    <limit name="totalGigabytesUsed" value="0"/>

to the xml response:
/api_samples/os-used-limits-for-admin/usedlimitsforadmin-get-resp.xml

PS:
http://api.openstack.org/api-ref-compute.html#compute_limits_admins
this method also return response in the same format, so it points to the same example file as above, change the above file will affect both methods

can someone clarify on this?

Tom Fifield (fifieldt)
tags: added: volume-api
removed: cinder
Revision history for this message
Miranda Zhang (miranda-zhang-q) wrote :

a bit more investigation with Compute v2 api

GET {host}:8774/v2/{tenantID}/limits

{"limits": {"rate": [], "absolute": {"maxServerMeta": 128, "maxPersonality": 5, "maxImageMeta": 128, "maxPersonalitySize": 10240, "maxSecurityGroupRules": 20, "maxTotalKeypairs": 100, "totalRAMUsed": 0, "totalInstancesUsed": 0, "maxSecurityGroups": 10, "totalFloatingIpsUsed": 0, "maxTotalCores": 20, "totalSecurityGroupsUsed": 0, "maxTotalFloatingIps": 10, "maxTotalInstances": 10, "totalCoresUsed": 0, "maxTotalRAMSize": 51200}}}

Can not observe any of the following additional params

totalVolumesUsed
totalGigabytesUsed

I suspect that this fix is for API version 1? so maybe no longer valid?

Changed in openstack-api-site:
assignee: Miranda Zhang (miranda-zhang-q) → Diane Fleming (diane-fleming)
status: Confirmed → In Progress
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to api-site (master)

Fix proposed to branch: master
Review: https://review.openstack.org/90611

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/90611
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=201b1327ceeda3f80042a1fdace69030e8cf1981
Submitter: Jenkins
Branch: master

commit 201b1327ceeda3f80042a1fdace69030e8cf1981
Author: Diane Fleming <email address hidden>
Date: Sun Apr 27 14:51:12 2014 -0500

    Add absolute limits extension for cinder

    Also, clean up code samples (extra copies)
    Update pointers to code samples
    Add parameter descriptions to common.ent for limits extension

    Closes-Bug: #1240789

    Change-Id: I8e2f4cc15392a0c560bddda9a20a8b0aa3e67c3b
    author: diane fleming

Changed in openstack-api-site:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.