api/config.py should remove some outdated settings

Bug #1408858 reported by ZhiQiang Fan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
ZhiQiang Fan

Bug Description

We have defined some options but never active them since we have no static files

https://github.com/openstack/ceilometer/blob/master/ceilometer/api/config.py#L23

We'd better remove them

Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

pecan need those options even they are not exist...

AttributeError: 'pecan.conf' object has no attribute 'template_path'

Changed in ceilometer:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
status: New → Invalid
ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
status: Invalid → In Progress
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/146285

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

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

commit 75b1c9a3aba389235be9d556b702103e3923a721
Author: ZhiQiang Fan <email address hidden>
Date: Sun Jan 11 02:48:11 2015 +0800

    Remove unused pecan configuration options

    The pecan configuration options `static_root` and `template_path`
    in ceilometer/api/config.py has never take effect since the files
    which have specified don't exist. The two options actually is not
    required according to:
    http://pecan.readthedocs.org/en/latest/configuration.html#application-configuration

    This patch removes the two pecan configuration options.

    Change-Id: If7e7b264145544c1ff5359d108581320ee942b4d
    Closes-Bug: #1408858

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