set default limit to meter/sample queries

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

Bug Description

https://review.openstack.org/196792
commit 1c546b10b88256c96467c541f725fd753397e09d
Author: gordon chung <email address hidden>
Date: Mon Jun 29 14:29:27 2015 -0400

    set default limit to meter/sample queries

    this patch makes existing limit options in meter and samples apis
    mandatory. if not set explicitly, a default limit will be applied.

    DocImpact
    ApiImpact

    Change-Id: Ie354e8e64a6f8677a0be7e75c34e8901cdffbc6a
    Implements: blueprint mandatory-limit

Tags: ceilometer
Revision history for this message
Lana (loquacity) wrote :

I can't see an openstack-manuals docimpact here. Please reopen with more info if I'm wrong.

Changed in openstack-manuals:
status: New → Incomplete
Changed in openstack-manuals:
status: Incomplete → Invalid
Revision history for this message
gordon chung (chungg) wrote :

this is a change in api. the results returned are limited. it also adds a new config option.

Changed in openstack-manuals:
status: Invalid → New
Revision history for this message
Gauvain Pocentek (gpocentek) wrote :

I missed the config option, sorry. It is already included in the config ref.

affects: openstack-manuals → openstack-api-site
Anne Gentle (annegentle)
Changed in openstack-api-site:
status: New → Confirmed
importance: Undecided → Medium
Changed in openstack-api-site:
assignee: nobody → Diane Fleming (diane-fleming)
milestone: none → liberty
Changed in openstack-manuals:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Diane Fleming (diane-fleming)
milestone: none → liberty
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/221097

Changed in openstack-manuals:
status: Confirmed → In Progress
Revision history for this message
Diane Fleming (diane-fleming) wrote :
Changed in openstack-api-site:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-manuals (master)

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

commit f9435e19b1a0ddd6595f1d80ef3402d84340c785
Author: Diane Fleming <email address hidden>
Date: Mon Sep 7 14:55:40 2015 -0500

    Update default_api_return_limit to 100, which is its actual value

    It showed an incorrect value in the config changes overview

    Change-Id: I7ac18b6ba53eb1460a0871e77204639849342618
    Closes-Bug: #1471197

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

Reviewed: https://review.openstack.org/221098
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=59296e54a8452ea20e0dc63f18ab62b75ad8abde
Submitter: Jenkins
Branch: master

commit 59296e54a8452ea20e0dc63f18ab62b75ad8abde
Author: Diane Fleming <email address hidden>
Date: Mon Sep 7 14:49:35 2015 -0500

    Add note about limits for meters and samples APIs

    Change-Id: I8ba1fd670db70e313da49d37142a8b70e42c0979
    Partial-Bug: #1471197

Changed in openstack-api-site:
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.