[queues] Stop using global config

Bug #1239725 reported by Kurt Griffiths on 2013-10-14
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
High
Kurt Griffiths

Bug Description

This is a prerequisite to sharding support, so that config can optionally be loaded from the catalog in lieu of the INI file.

Kurt Griffiths (kgriffs) on 2013-10-14
Changed in marconi:
status: New → Triaged
status: Triaged → In Progress
importance: Undecided → High
assignee: nobody → Kurt Griffiths (kgriffs)
Allele Dev (alleledev) on 2013-10-14
summary: - Stop using global config
+ [queues] Stop using global config

Reviewed: https://review.openstack.org/51705
Committed: http://github.com/openstack/marconi/commit/c67d5a4a2516644e17740ab38ef71c870a470ba4
Submitter: Jenkins
Branch: master

commit c67d5a4a2516644e17740ab38ef71c870a470ba4
Author: kgriffs <email address hidden>
Date: Mon Oct 14 09:51:48 2013 -0500

    fix(queues): Global config used everywhere

    This patch modifies the queues code to use manually instantiated configs
    rather than the global config. This will allow for more flexibility in
    configuring components. For example, the up-and-coming sharding feature
    can construct configs based on the shard catalog rather than the local
    INI file. Also, this change will make testing easier.

    Also in this patch, the SQLite driver schema was consolidated into
    one place in preparation for the sharding patch. With this change,
    controllers can be instantiated and used independently, since all
    the tables are set up when the driver is loaded, not in a piecemeal
    fashion as before.

    Change-Id: I1afa8ab8c7e6dd9a017f4a9e3a3b1fadbeb32806
    Implements: blueprint remove-global-config
    Closes-Bug: #1239725

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-12-19
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers