[telemetry] os_auth_url should use identity_endpoint in template

Bug #1479399 reported by Yi Ming Yin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Fix Released
Medium
Yi Ming Yin

Bug Description

In current telemetry cookbook, template ceilometer.conf.erb is using auth_uri for following both options:

[DEFAULT]
os_auth_url = <%= @auth_uri %>

[keystone_authtoken]
auth_uri = <%= @auth_uri %>

However, the first os_auth_url should use identity_endpoint for connecting to keystone.

https://github.com/openstack/ceilometer/blob/e71e391a4f787e3875862b71cb1377537ef1a97e/ceilometer/service.py#L72

 cfg.StrOpt('os-auth-url',
               deprecated_group="DEFAULT",
               default=os.environ.get('OS_AUTH_URL',
                                      'http://localhost:5000/v2.0'),

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

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

Changed in openstack-chef:
assignee: nobody → Yi Ming Yin (yinym-m)
status: New → In Progress
description: updated
Changed in openstack-chef:
importance: Undecided → Medium
milestone: none → kilo-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-telemetry (master)

Reviewed: https://review.openstack.org/207084
Committed: https://git.openstack.org/cgit/openstack/cookbook-openstack-telemetry/commit/?id=76ba93e3623bcf755e5f1ba61bac0da85f0688a4
Submitter: Jenkins
Branch: master

commit 76ba93e3623bcf755e5f1ba61bac0da85f0688a4
Author: Yi Ming Yin <email address hidden>
Date: Wed Jul 29 23:04:47 2015 +0800

    Change os_auth_url to use identity_endpoint in conf template

    Current template ceilometer.conf.erb is using auth_uri for os_auth_url
    which is not a correct approach. This auth_uri is dedicated to be used
    by auth_uri under section [keystone_authtoken]. On the other hand, the
    passed in identity_endpoint param is not used. Actually, by comparison
    with other cookbooks, identity_endpoint should be used for keystone
    identity service connection.

    Change-Id: I50c631000efe9c82cd83723c1bf9a7c6ef54556c
    Closes-bug: #1479399

Changed in openstack-chef:
status: In Progress → Fix Released
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.