rate limits are not returned by /info API

Bug #1308989 reported by Florent Flament
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Undecided
paul luse

Bug Description

While rate limiting works properly, when setting container_ratelimit and container_listing_ratelimit, it doesn't appear in the json returned by the /info API.

ubuntu@devstack:~$ grep _ratelimit /etc/swift/proxy-server.conf
# account_ratelimit of 0 means disabled
# account_ratelimit = 0
container_ratelimit_0 = 1
container_ratelimit_10 = 1
container_ratelimit_50 = 1
container_listing_ratelimit_0 = 1
container_listing_ratelimit_10 = 1
container_listing_ratelimit_50 = 1

ubuntu@devstack:~$ curl -s http://localhost:8080/info | python -m json.tool
{
    "account_quotas": {},
    "bulk_delete": {
        "max_deletes_per_request": 10000,
        "max_failed_deletes": 1000
    },
    "bulk_upload": {
        "max_containers_per_extraction": 10000,
        "max_failed_extractions": 1000
    },
    "container_quotas": {},
    "container_sync": {
        "realms": {}
    },
    "crossdomain": {},
    "formpost": {},
    "keystoneauth": {},
    "ratelimit": {},
    "slo": {
        "max_manifest_segments": 1000,
        "max_manifest_size": 2097152,
        "min_segment_size": 1048576
    },
    "staticweb": {},
    "swift": {
        "account_listing_limit": 10000,
        "container_listing_limit": 10000,
        "max_account_name_length": 256,
        "max_container_name_length": 256,
        "max_file_size": 5368709122,
        "max_header_size": 8192,
        "max_meta_count": 90,
        "max_meta_name_length": 128,
        "max_meta_overall_size": 4096,
        "max_meta_value_length": 256,
        "max_object_name_length": 1024,
        "strict_cors_mode": true,
        "version": "1.13.1.rc1.35.g77b8d42"
    },
    "tempauth": {
        "account_acls": true
    },
    "tempurl": {
        "methods": [
            "GET",
            "HEAD",
            "PUT"
        ]
    }
}

paul luse (paul-e-luse)
Changed in swift:
assignee: nobody → paul luse (paul-e-luse)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to swift (master)

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

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

Reviewed: https://review.openstack.org/88633
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=408588389576f5a239651034bee25c0961b12c81
Submitter: Jenkins
Branch: master

commit 408588389576f5a239651034bee25c0961b12c81
Author: Paul Luse <email address hidden>
Date: Fri Apr 18 10:16:11 2014 -0700

    Add Ratelimit parameters to /info

    We previously registered with no parameters, added parms so they
    are displayed as follows (example):

    "ratelimit": {"max_sleep_time_seconds": 60.0,
    "container_listing_ratelimits": [[0, 100.0], [10, 50.0], [50, 20.0]],
    "container_ratelimits": [[0, 100.0], [10, 50.0], [50, 20.0]],
    "account_ratelimit": 1.0}

    Note that not all parameters are exposed (intentionally) via /info

    Change-Id: I36c7ef15af17e3eb8ebb93429035bd06d089a945
    Closes-Bug: 1308989

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