ImportError for missing Oslo module openstack.common.log_handler

Bug #1244722 reported by Brian Cline
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Brian Cline

Bug Description

Oslo commit b1352340 moved PublishErrorsHandler into log_handler.py. Ceilometer has the updated log.py module, but not log_handler.

As a result, using publish_errors causes ceilometer-api, etc. to throw an ImportError at startup:

root@dal00b-control:/etc/ceilometer# ceilometer-api
Traceback (most recent call last):
  File "/usr/bin/ceilometer-api", line 10, in <module>
    sys.exit(start())
  File "/usr/lib/python2.7/dist-packages/ceilometer/api/app.py", line 110, in start
    service.prepare_service()
  File "/usr/lib/python2.7/dist-packages/ceilometer/service.py", line 95, in prepare_service
    log.setup('ceilometer')
  File "/usr/lib/python2.7/dist-packages/ceilometer/openstack/common/log.py", line 359, in setup
    _setup_logging_from_conf()
  File "/usr/lib/python2.7/dist-packages/ceilometer/openstack/common/log.py", line 422, in _setup_logging_from_conf
    logging.ERROR)
  File "/usr/lib/python2.7/dist-packages/ceilometer/openstack/common/importutils.py", line 40, in import_object
    return import_class(import_str)(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/ceilometer/openstack/common/importutils.py", line 30, in import_class
    __import__(mod_str)
ImportError: No module named log_handler

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/53906

Changed in ceilometer:
assignee: nobody → Brian Cline (briancline)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/53906
Committed: http://github.com/openstack/ceilometer/commit/f181ece2fd1e0cb90b0453fea2a6c5fcf5d4b82d
Submitter: Jenkins
Branch: master

commit f181ece2fd1e0cb90b0453fea2a6c5fcf5d4b82d
Author: Brian Cline <email address hidden>
Date: Fri Oct 25 11:32:30 2013 -0500

    Sync log_handler from Oslo

    Resolves an ImportError when enabling publish_errors for
    Ceilometer services. Essentially, PublishErrorsHandler was
    moved in Oslo from *.log to *.log_handler, but log_handler
    was not yet being pulled in.

    Oslo commit: Ibf28ba17f81b5ddeda653f6a3501ecac18ffcd42

    Change-Id: I23bc7fd0ea4c811e133e743634032fb836c34b22
    Fixes: Bug #1244722

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