Heat set the incorrect ca file argument for ceilometer client

Bug #1391792 reported by zhu zhu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Undecided
Jeff Peeler

Bug Description

When ceilometer is configured with SSL, found heat fail to connect to it using ca cert.

Found that the code below have incorrect argument set. 'ca_file' should be 'cacert' instead.

https://github.com/openstack/heat/blob/master/heat/engine/clients/os/ceilometer.py#L37

See ceilometerclient code:
https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/client.py#L53

Changed in heat:
assignee: nobody → Jeff Peeler (jpeeler-z)
status: New → In Progress
Revision history for this message
Jeff Peeler (jpeeler-z) wrote :

Not sure I understand why this got assigned to me. The review is in progress here:

https://review.openstack.org/134158

Revision history for this message
Jeff Peeler (jpeeler-z) wrote :

I forgot that I updated the commit message.

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

Reviewed: https://review.openstack.org/134158
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=72d0313fab470534d2cffb6c7b8a6c311d68740c
Submitter: Jenkins
Branch: master

commit 72d0313fab470534d2cffb6c7b8a6c311d68740c
Author: ZHU ZHU <email address hidden>
Date: Thu Nov 13 04:39:04 2014 -0600

    Fix incorrect ca argument for heat ceilometer client

    Currently Heat ceilometer client are passing the wrong ca argument
    during client instantiation. It will cause heat fail to connect to ceilometer
    if it's configured with SSL.

    https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/client.py#L53

    Closes-bug: #1391792

    Change-Id: Ib32dfede0ece78bc39e0cab213cb51df8eb62eb6

Changed in heat:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in heat:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: kilo-1 → 2015.1.0
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.