[telemetry] os_auth_url should use identity_endpoint in template

Bug #1479399 reported by Yi Ming Yin on 2015-07-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
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'),

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

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  Edit
Everyone can see this information.

Other bug subscribers