Ceilometer expirer reports NotSupportedWarning for Database related configuration options

Bug #1516515 reported by Gautam Prasad
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Aodh
Fix Released
Low
ZhiQiang Fan
Ceilometer
Fix Released
Low
ZhiQiang Fan

Bug Description

The below exception gets thrown in ceilometer-expirer logs.

/usr/lib/python2.7/site-packages/oslo_db/sqlalchemy/enginefacade.py:241: NotSupportedWarning: Configuration option(s) ['alarm_connection', 'alarm_history_time_to_live', 'db2nosql_resource_id_maxlen', 'event_connection', 'event_time_to_live', 'metering_connection', 'metering_time_to_live'] not supported exception.NotSupportedWarning

From what I understand, enginefacade.py takes in sqlalchemy specific database configuration options. For ex. look at the __init__() method in oslo_db/sqlalchemy/engine_facade.py. Ceilometer specifies some the options like "metering_time_to_live" etc. as part of database section in ceilometer.conf. These are needed by ceilometer and are not deprecated/unsupported. These are in some way related to Ceilometer Database, but are not actual database related sqlalchemy configuration options. Hence the default behaviour of engine_facade is to log warning about these.

Revision history for this message
Gautam Prasad (gautpras) wrote :

I am using OpenStack Liberty build.

ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
importance: Undecided → Low
status: New → Confirmed
ZhiQiang Fan (aji-zqfan)
Changed in aodh:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
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/245744

Changed in ceilometer:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to aodh (master)

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

Changed in aodh:
status: New → In Progress
ZhiQiang Fan (aji-zqfan)
Changed in aodh:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to aodh (master)

Reviewed: https://review.openstack.org/245748
Committed: https://git.openstack.org/cgit/openstack/aodh/commit/?id=2b3e3f0de6a01080600263b436df484273063267
Submitter: Jenkins
Branch: master

commit 2b3e3f0de6a01080600263b436df484273063267
Author: ZhiQiang Fan <email address hidden>
Date: Tue Nov 17 04:38:27 2015 +0800

    don't pass aodh options to oslo.db engine facade

    oslo.db EngineFacade class only support options defined by oslo.db,
    but we also pass options defined by aodh, so oslo.db complains
    with a warning message:

    NotSupportedWarning: Configuration option(s) ['alarm_connection', \
    'alarm_history_time_to_live'] not supported

    This patch makes sure only options defined by oslo.db will be sent
    to EngineFacade

    Change-Id: I4c259f29af1cab60c3aee41a610b12f64ed5f2d5
    Closes-Bug: #1516515

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

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

commit 43c65a3b658c0af03948780ab0452b5ec2d05963
Author: ZhiQiang Fan <email address hidden>
Date: Tue Nov 17 00:34:16 2015 +0800

    don't pass ceilometer options to oslo.db engine facade

    oslo.db EngineFacade class only support options defined by oslo.db,
    but we're passing options defined by ceilometer, so oslo.db complains
    with a warning message:

    NotSupportedWarning: Configuration option(s) ['alarm_connection', \
    'alarm_history_time_to_live', 'db2nosql_resource_id_maxlen', \
    'event_connection', 'event_time_to_live', 'metering_connection', \
    'metering_time_to_live'] not supported exception.NotSupportedWarning

    This patch makes sure only options defined by oslo.db will be sent
    to EngineFacade

    Change-Id: I2a39737a5f7824918c28ff2180f2ea95e7e82537
    Closes-Bug: #1516515

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/249921

Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/ceilometer 6.0.0.0b1

This issue was fixed in the openstack/ceilometer 6.0.0.0b1 development milestone.

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/aodh 2.0.0.0b1

This issue was fixed in the openstack/aodh 2.0.0.0b1 development milestone.

Changed in aodh:
status: Fix Committed → Fix Released
Liusheng (liusheng)
Changed in aodh:
milestone: none → next-mitaka
Changed in ceilometer:
milestone: none → mitaka-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/liberty)

Reviewed: https://review.openstack.org/249921
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=40c5ceefa96d93cf9b12fa189787ed3f26e853cc
Submitter: Jenkins
Branch: stable/liberty

commit 40c5ceefa96d93cf9b12fa189787ed3f26e853cc
Author: ZhiQiang Fan <email address hidden>
Date: Tue Nov 17 00:34:16 2015 +0800

    don't pass ceilometer options to oslo.db engine facade

    oslo.db EngineFacade class only support options defined by oslo.db,
    but we're passing options defined by ceilometer, so oslo.db complains
    with a warning message:

    NotSupportedWarning: Configuration option(s) ['alarm_connection', \
    'alarm_history_time_to_live', 'db2nosql_resource_id_maxlen', \
    'event_connection', 'event_time_to_live', 'metering_connection', \
    'metering_time_to_live'] not supported exception.NotSupportedWarning

    This patch makes sure only options defined by oslo.db will be sent
    to EngineFacade

    Note: Ceilometer stable/kilo still has alarm code, so this backport
    also contains change in Aodh project.

    Change-Id: I2a39737a5f7824918c28ff2180f2ea95e7e82537
    Closes-Bug: #1516515
    (cherry picked from commit 43c65a3b658c0af03948780ab0452b5ec2d05963)

tags: added: in-stable-liberty
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/ceilometer 5.0.2

This issue was fixed in the openstack/ceilometer 5.0.2 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to aodh (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/275811

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to aodh (stable/liberty)

Reviewed: https://review.openstack.org/275811
Committed: https://git.openstack.org/cgit/openstack/aodh/commit/?id=5901c3a87982367d2008d0c048c821ef5e471e37
Submitter: Jenkins
Branch: stable/liberty

commit 5901c3a87982367d2008d0c048c821ef5e471e37
Author: ZhiQiang Fan <email address hidden>
Date: Tue Nov 17 04:38:27 2015 +0800

    don't pass aodh options to oslo.db engine facade

    oslo.db EngineFacade class only support options defined by oslo.db,
    but we also pass options defined by aodh, so oslo.db complains
    with a warning message:

    NotSupportedWarning: Configuration option(s) ['alarm_connection', \
    'alarm_history_time_to_live'] not supported

    This patch makes sure only options defined by oslo.db will be sent
    to EngineFacade

    Change-Id: I4c259f29af1cab60c3aee41a610b12f64ed5f2d5
    Closes-Bug: #1516515
    (cherry picked from commit 2b3e3f0de6a01080600263b436df484273063267)

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/aodh 1.1.2

This issue was fixed in the openstack/aodh 1.1.2 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.