Patch to openstack.common.cfg should be removed

Bug #992319 reported by Brian Waldon on 2012-05-01
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
High
Mark McLoughlin

Bug Description

Glance master deviates from the official openstack.common.cfg:find_config_files; it looks in a local 'etc' dir. This needs to be extracted into openstack-common.

Brian Waldon (bcwaldon) on 2012-05-07
Changed in glance:
assignee: nobody → Brian Waldon (bcwaldon)
status: Triaged → In Progress
assignee: Brian Waldon (bcwaldon) → Mark McLoughlin (markmc)
milestone: none → folsom-1
Changed in glance:
assignee: Mark McLoughlin (markmc) → Brian Waldon (bcwaldon)

Reviewed: https://review.openstack.org/6957
Committed: http://github.com/openstack/glance/commit/db86220abb0861ccb58fb6af57a262c734a18d6e
Submitter: Jenkins
Branch: master

commit db86220abb0861ccb58fb6af57a262c734a18d6e
Author: Mark McLoughlin <email address hidden>
Date: Tue May 1 10:30:03 2012 +0100

    Use ConfigOpts.find_file() for policy and paste

    Use cfg's new helper method to find the PasteDeploy and policy config
    files. The basic behavior is "look alongside the config file" with a
    fall back to the standard default config paths.

    In the fairly obscure case where no --config-file is found we use an
    explicit default paste file taking to account the fact that all the
    glance apps have their paste config in a single file.

    Add some new debug logging to make it easier to see which config files
    are being loaded.

    Fixes bug 992319

    Change-Id: I978c4a8cb6ec6d12aaf746af511ccb572c34b7ee

Changed in glance:
status: In Progress → Fix Committed
Brian Waldon (bcwaldon) on 2012-05-07
Changed in glance:
assignee: Brian Waldon (bcwaldon) → Mark McLoughlin (markmc)
Thierry Carrez (ttx) on 2012-05-23
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-09-27
Changed in glance:
milestone: folsom-1 → 2012.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers