Ceilometer

API ACL authentication is broken

Bug #1098204 reported by Julien Danjou
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Critical
Julien Danjou
Ceilometer 2013.1 "grizzly"

Bug Description

The ceilometer-api ACL authentication is broken. The configuration is nor read nor passed properly to the keystone middleware, likely because of recent changes in it.

Julien Danjou (jdanjou)
Changed in ceilometer:
assignee: nobody → Julien Danjou (jdanjou)
importance: Undecided → Critical
status: New → Confirmed
milestone: none → grizzly-2
Revision history for this message
Nick Barcet (nijaba) wrote :

https://review.openstack.org/#/c/19390/

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

Reviewed: https://review.openstack.org/19390
Committed: http://github.com/openstack/ceilometer/commit/092aad40bd290dda7898a006b392136e63f397fa
Submitter: Jenkins
Branch: master

commit 092aad40bd290dda7898a006b392136e63f397fa
Author: Julien Danjou <email address hidden>
Date: Thu Jan 10 15:21:11 2013 +0100

    Fix keystoneclient auth_token middleware changes

    Things changed, and the configuration wasn't read anymore. This patch fixes
    that. It also remove duplicated code in acl.py, so the problem is fixed only
    in one place. Finally, it uses prepare_service() to find the right
    configuration file for ceilometer.

    This fixes bug #1098204

    Change-Id: I0d6c30ad443a4d0db201e60d12b275625a4bee6e
    Signed-off-by: Julien Danjou <email address hidden>

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (milestone-proposed)

Fix proposed to branch: milestone-proposed
Review: https://review.openstack.org/19456

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

Reviewed: https://review.openstack.org/19456
Committed: http://github.com/openstack/ceilometer/commit/269aa99b6fe30d2736e92ce7dc8cd05db6a2bf4c
Submitter: Jenkins
Branch: milestone-proposed

commit 269aa99b6fe30d2736e92ce7dc8cd05db6a2bf4c
Author: Julien Danjou <email address hidden>
Date: Thu Jan 10 15:21:11 2013 +0100

    Fix keystoneclient auth_token middleware changes

    Things changed, and the configuration wasn't read anymore. This patch fixes
    that. It also remove duplicated code in acl.py, so the problem is fixed only
    in one place. Finally, it uses prepare_service() to find the right
    configuration file for ceilometer.

    This fixes bug #1098204

    Change-Id: I0d6c30ad443a4d0db201e60d12b275625a4bee6e
    Signed-off-by: Julien Danjou <email address hidden>

Changed in ceilometer:
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.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.