Nova rate limit defaults

Bug #1299907 reported by Belmiro Moreira
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-api-site
Fix Released
Medium
Andreas Jaeger
openstack-manuals
Fix Released
Medium
Andreas Jaeger

Bug Description

In the docs:
http://docs.openstack.org/trunk/config-reference/content/configuring-compute-API.html
http://docs.openstack.org/api/openstack-compute/2/content/Rate_Limits-d1e862.html

nova default rate limits are not updated considering:
https://review.openstack.org/#/c/34774/

I'm confused because there was a doc bug merged for havana "Sane rest API rate limit defaults" that should have fixed this.
https://bugs.launchpad.net/openstack-manuals/+bug/1195433

Revision history for this message
Anne Gentle (annegentle) wrote : Re: [Bug 1299907] [NEW] Nova rate limit defaults

The openstack-manuals repo does not contain the file for
http://docs.openstack.org/api/openstack-compute/2/content/Rate_Limits-d1e862.html

It does contain the file for
http://docs.openstack.org/trunk/config-reference/content/configuring-compute-API.html

What would you like to change in the compute-api repo to fix the first link?

On Sun, Mar 30, 2014 at 4:08 PM, Belmiro Moreira <<email address hidden>
> wrote:

> Public bug reported:
>
> In the docs:
>
> http://docs.openstack.org/trunk/config-reference/content/configuring-compute-API.html
>
> http://docs.openstack.org/api/openstack-compute/2/content/Rate_Limits-d1e862.html
>
> nova default rate limits are not updated considering:
> https://review.openstack.org/#/c/34774/
>
>
> I'm confused because there was a doc bug merged for havana "Sane rest API
> rate limit defaults" that should have fixed this.
> https://bugs.launchpad.net/openstack-manuals/+bug/1195433
>
> ** Affects: openstack-manuals
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are subscribed to
> openstack-manuals.
> https://bugs.launchpad.net/bugs/1299907
>
> Title:
> Nova rate limit defaults
>
> Status in OpenStack Manuals:
> New
>
> Bug description:
> In the docs:
>
> http://docs.openstack.org/trunk/config-reference/content/configuring-compute-API.html
>
> http://docs.openstack.org/api/openstack-compute/2/content/Rate_Limits-d1e862.html
>
> nova default rate limits are not updated considering:
> https://review.openstack.org/#/c/34774/
>
>
> I'm confused because there was a doc bug merged for havana "Sane rest
> API rate limit defaults" that should have fixed this.
> https://bugs.launchpad.net/openstack-manuals/+bug/1195433
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/openstack-manuals/+bug/1299907/+subscriptions
>

Revision history for this message
Belmiro Moreira (moreira-belmiro-email-lists) wrote :

Looking into the code the new values are:

DEFAULT_LIMITS = [
    Limit("POST", "*", ".*", 120, utils.TIME_UNITS['MINUTE']),
    Limit("POST", "*/servers", "^/servers", 120, utils.TIME_UNITS['MINUTE']),
    Limit("PUT", "*", ".*", 120, utils.TIME_UNITS['MINUTE']),
    Limit("GET", "*changes-since*", ".*changes-since.*", 120,
          utils.TIME_UNITS['MINUTE']),
    Limit("DELETE", "*", ".*", 120, utils.TIME_UNITS['MINUTE']),
    Limit("GET", "*/os-fping", "^/os-fping", 12, utils.TIME_UNITS['MINUTE']),
]

Revision history for this message
Tom Fifield (fifieldt) wrote :

To fix this bug, make the edits in: ../common/section_compute_config-api.xml

Changed in openstack-manuals:
status: New → Triaged
milestone: none → icehouse
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-manuals (master)

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

Changed in openstack-manuals:
assignee: nobody → Andreas Jaeger (jaegerandi)
status: Triaged → In Progress
Revision history for this message
Andreas Jaeger (jaegerandi) wrote :

Patch for compute-api:
https://review.openstack.org/85593

Changed in openstack-api-site:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Andreas Jaeger (jaegerandi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to compute-api (master)

Reviewed: https://review.openstack.org/85593
Committed: https://git.openstack.org/cgit/openstack/compute-api/commit/?id=1680667e17aad52f7761a8ead7b01f7dc5e5a5ac
Submitter: Jenkins
Branch: master

commit 1680667e17aad52f7761a8ead7b01f7dc5e5a5ac
Author: Andreas Jaeger <email address hidden>
Date: Sun Apr 6 14:06:22 2014 +0200

    Update default rate limits for nova

    Update limits to current limits in
    nova/api/openstack/compute/limits.py.
    Remove sentence that dos not apply anymore.
    Remove unneeded colspan elements, they're the same everywhere.

    Fix reference to make this build after changs in api-site:
    it's now #serverMetadataKey

    Change-Id: Ib225533aaba9ad4250ebefe0707b87f2ea8dcb66
    Closes-Bug: #1299907

Changed in openstack-api-site:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-manuals (master)

Reviewed: https://review.openstack.org/85586
Committed: https://git.openstack.org/cgit/openstack/openstack-manuals/commit/?id=8cedd581f5de2bc9f726f1383117d0f137206a9c
Submitter: Jenkins
Branch: master

commit 8cedd581f5de2bc9f726f1383117d0f137206a9c
Author: Andreas Jaeger <email address hidden>
Date: Sun Apr 6 13:15:21 2014 +0200

    Update default rate limits for nova

    Update limits to current limits in nova/api/openstack/compute/limits.py.
    Remove sentence that dos not apply anymore.

    Change-Id: Ib67c25731871a29eae38caa99af38ac2dd341a73
    Closes-Bug: #1299907

Changed in openstack-manuals:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-manuals 15.0.0

This issue was fixed in the openstack/openstack-manuals 15.0.0 release.

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.