Keystone middleware does not use configuration

Bug #1071047 reported by Julien Danjou
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
Doug Hellmann

Bug Description

The configuration file and its value for the Keystone middleware does not seem to be used by it, and therefore the default value are always used.

Revision history for this message
Steven Berler (steven-berler) wrote :

I ran into this when trying to run ceilometer on devstack.
The ceilometer-api in devstack currently doesn't use a --config-file command-line option, so I tried adding that as well as creating a new config file for it in /etc/ceilometer/ceilometer-api.conf, but it doesn't seem to be using those values for the keystone middleware.

Revision history for this message
Julien Danjou (jdanjou) wrote :

I've already sent a patch for devstack at https://review.openstack.org/#/c/14766/

But as you state, this bug is really about the config file not being taken into account. :(

Changed in ceilometer:
assignee: Julien Danjou (jdanjou) → Doug Hellmann (doug-hellmann)
status: Confirmed → In Progress
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

There are a couple of issues. First, we're initializing the middleware before we read the config file, but then we're passing an empty dictionary to the middleware instead of the config object. I have a working version, but I need to figure out how to package it so it will work with mod_wsgi or another wsgi container.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Actually, I'm going to worry about the mod_wsgi stuff as a separate issue, since that's not really related and it didn't work already.

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

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

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

Reviewed: https://review.openstack.org/15133
Committed: http://github.com/stackforge/ceilometer/commit/085fa79ec0240751be1bbd3c0025097970319a8e
Submitter: Jenkins
Branch: master

commit 085fa79ec0240751be1bbd3c0025097970319a8e
Author: Doug Hellmann <email address hidden>
Date: Wed Oct 31 15:24:42 2012 -0400

    Fix auth middleware configuration

    Initialze the configuration object before initializing
    the middleware, and pass the resulting config in to
    the middleware so it can get the configuration settings.

    addresses bug #1071047

    Change-Id: I2a487d2a2f2d3467e522868ac78dc4645bf7d643
    Signed-off-by: Doug Hellmann <email address hidden>

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