Remove ratelimit from v3 api

Bug #1255471 reported by Alex Xu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Alex Xu

Bug Description

The ratelimit middleware isn't really useful. that point out by https://review.openstack.org/#/c/34821/

So we can remove ratelimit from v3 api. And after discussion from mail-list:
http://lists.openstack.org/pipermail/openstack-dev/2013-November/020291.html

there isn't any one and any reason object it.

Tags: api
Alex Xu (xuhj)
Changed in nova:
assignee: nobody → Alex Xu (xuhj)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Alex Xu (xuhj)
tags: added: api
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/58713
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=93d3e47531942243676342e10f569cda396a42c5
Submitter: Jenkins
Branch: master

commit 93d3e47531942243676342e10f569cda396a42c5
Author: He Jie Xu <email address hidden>
Date: Wed Nov 27 17:32:28 2013 +0800

    Remove middleware ratelimits from v3 api

    The ratelimit middleware isn't really useful. That is pointed out
    by https://review.openstack.org/#/c/34821/. And it didn't get any
    object from mail-list:
    http://lists.openstack.org/pipermail/openstack-dev/2013-November/020291.html

    This patch remove ratelimit middleware and limits extension. The config option
    'api_rate_limit' is only valid in v2 api. The pipeline factory for v3 api won't
    check that option anymore. For compatibility, if user is still using old
    'api-paste.init', pipeline factory will ignore to load ratelimit middleware, and
    print warning message to user for notice ratelimit is deprecated in v3 api.

    Closes-bug: 1255471
    DocImpact

    Change-Id: Ifeae0504e11089f95e4d8af58bcb7372dca87f81

Changed in nova:
status: In Progress → Fix Committed
Changed in nova:
milestone: none → icehouse-2
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-2 → 2014.1
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.