Limit value None in the message controller in a pooled marconi deployment

Bug #1324587 reported by Prashanth Raghu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
High
Nataliia Uvarova

Bug Description

Limit values are appearing as None in place of the default storage values provided in the storage driver when marconi is used in a pooled context.

Was observed in the : message and queue controllers.

Revision history for this message
Allele Dev (alleledev) wrote :

Does the affect the behavior of the message controller in a sharded context, i.e., is the user able to exceed the default limit or the configuration time limit?

Kurt Griffiths (kgriffs)
summary: - Limit value None in the message controller in a sharded marconi
+ Limit value None in the message controller in a pooled marconi
deployment
description: updated
Kurt Griffiths (kgriffs)
Changed in marconi:
status: New → Triaged
importance: Undecided → Medium
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to marconi (master)

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

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

Reviewed: https://review.openstack.org/102457
Committed: https://git.openstack.org/cgit/openstack/marconi/commit/?id=b3274e3857369b2cd29fbf00821863116ca905f9
Submitter: Jenkins
Branch: master

commit b3274e3857369b2cd29fbf00821863116ca905f9
Author: Nataliia Uvarova <email address hidden>
Date: Wed Jun 25 10:26:57 2014 +0300

    Make storage/pooling reflect storage/base

    Controllers wrappers for pooled context wasn't fully compatible with the
    APIs defined in storage/base. Order of parameters was different and
    default values was not specified or specified incorrectly. In
    particular in pooled setup default limits were None in message and claim
    controllers. This commit changes them to be the same as in non-pooled
    setup.

    Also tests for testing default limits were added.

    Change-Id: I2b83383975986179813df7a897d78852a532564e
    Closes-bug: #1324587

Changed in marconi:
status: In Progress → Fix Committed
no longer affects: marconi/juno
Thierry Carrez (ttx)
Changed in zaqar:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in zaqar:
milestone: juno-3 → 2014.2
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.