connection can fail if database.connection is not defined

Bug #1419962 reported by gordon chung
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
gordon chung

Bug Description

we added support for a split db environment (metering_connection, event_connection...) but still support database.connection as our fall back option.

there's a bug in our connection logic where if you don't define a purpose it will assume you are connect to metering db but it only defaults to use database.connection option and does not check metering_connection option.

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

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

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

Reviewed: https://review.openstack.org/154222
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=f80cd71d86011b968bd141161369fd9ce27ddcd6
Submitter: Jenkins
Branch: master

commit f80cd71d86011b968bd141161369fd9ce27ddcd6
Author: gordon chung <email address hidden>
Date: Mon Feb 9 15:07:04 2015 -0500

    check metering_connection attribute by default

    ensure we check metering_connection and only default to
    database.connection if missing.

    Change-Id: I985f66c145bc8795bc93f846413823c4fea2a6ac
    Closes-Bug: #1419962

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → kilo-3
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: kilo-3 → 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.