Authentication error will be thrown if we construct ceilometer client in a certain way

Bug #1519729 reported by Xiangjun Li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Status tracked in Liberty
Kilo
New
Undecided
Xiangjun Li
Liberty
Fix Released
Undecided
Xiangjun Li
Mitaka
Fix Released
Undecided
Xiangjun Li

Bug Description

If we construct ceilometer client using the following way (https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/client.py#L388):
        from ceilometerclient import client as cclient
        connection = cclient.get_client(
            ‘2’,
            os_token={token},
           os_endpoint={endpoint})

This instance of ceilometer client will throw authentication error when use it later (e.g. call connection.events.list())

Xiangjun Li (lixiangjun)
Changed in python-ceilometerclient:
assignee: nobody → lixiangjun (lixiangjun)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ceilometerclient (master)

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

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

Reviewed: https://review.openstack.org/249690
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=b398841ebf1bb8acd0fea7c88dae613dad788d08
Submitter: Jenkins
Branch: master

commit b398841ebf1bb8acd0fea7c88dae613dad788d08
Author: xiangjun li <email address hidden>
Date: Wed Nov 25 18:08:51 2015 +0800

    Token will be lost when constructing a ceilometer client

    If parameters os_token and os_endpoint were passed to
    ceilometerclient.client.get_client to construct a ceilometer client, the
    token will be wrongly replaced with None in method
    ceilometerclient.client._adjust_kwargs, which will cause the
    authentication error when use the instance of ceilometer client later.
    This patch fixed this problem.

    Change-Id: Ib15cdd73b62b7f0b86d0bd6f19d344fd24a665f6
    Closes-Bug: #1519729

Changed in python-ceilometerclient:
status: In Progress → Fix Committed
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/python-ceilometerclient 2.1.0

This issue was fixed in the openstack/python-ceilometerclient 2.1.0 release.

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.