Ceilometer services don't start due to broken oslo_db requirement

Bug #1624692 reported by Paulo Matias
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Julien Danjou
OpenStack-Ansible
Fix Released
High
Jesse Pretorius

Bug Description

In Newton RC1, the Ceilometer services do not start. The regression was introduced by https://github.com/openstack/ceilometer/commit/1a53514b269963028bf6082002aa32e5a3d2f842

oslo_db seems to be imported unconditionally. Even if we are not using postgresql nor mysql, it tries to import oslo_db:

$ /openstack/venvs/ceilometer-14.0.0/bin/ceilometer-polling --polling-namespaces compute --log-file=/var/log/ceilometer/ceilometer-polling.log
Traceback (most recent call last):
  File "/openstack/venvs/ceilometer-14.0.0/bin/ceilometer-polling", line 7, in <module>
    from ceilometer.cmd.polling import main
  File "/openstack/venvs/ceilometer-14.0.0/lib/python2.7/site-packages/ceilometer/cmd/polling.py", line 21, in <module>
    from ceilometer.agent import manager
  File "/openstack/venvs/ceilometer-14.0.0/lib/python2.7/site-packages/ceilometer/agent/manager.py", line 38, in <module>
    from ceilometer import pipeline
  File "/openstack/venvs/ceilometer-14.0.0/lib/python2.7/site-packages/ceilometer/pipeline.py", line 33, in <module>
    from ceilometer.event.storage import models
  File "/openstack/venvs/ceilometer-14.0.0/lib/python2.7/site-packages/ceilometer/event/storage/models.py", line 18, in <module>
    from ceilometer.storage import base
  File "/openstack/venvs/ceilometer-14.0.0/lib/python2.7/site-packages/ceilometer/storage/__init__.py", line 19, in <module>
    from oslo_db import options as db_options
ImportError: No module named oslo_db

Manually doing a "pip install oslo.db" solves the issue.

Changed in openstack-ansible:
importance: Undecided → High
assignee: nobody → Kevin Carter (kevin-carter)
status: New → Confirmed
Changed in openstack-ansible:
assignee: Kevin Carter (kevin-carter) → Steve Lewis (steve-lewis)
status: Confirmed → In Progress
Changed in ceilometer:
assignee: nobody → venkatamahesh (venkatamaheshkotha)
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/372010

Changed in ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ceilometer:
assignee: venkatamahesh (venkatamaheshkotha) → Julien Danjou (jdanjou)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (master)

Change abandoned by venkatamahesh (<email address hidden>) on branch: master
Review: https://review.openstack.org/372010
Reason: Submitted by some other

Changed in openstack-ansible:
assignee: Steve Lewis (steve-lewis) → Jesse Pretorius (jesse-pretorius)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_ceilometer (master)

Reviewed: https://review.openstack.org/371955
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_ceilometer/commit/?id=ea89da7febdd71ef21d6634bc16fae88013b0dc3
Submitter: Jenkins
Branch: master

commit ea89da7febdd71ef21d6634bc16fae88013b0dc3
Author: Kevin Carter <email address hidden>
Date: Sat Sep 17 11:37:51 2016 -0500

    Add OSLO DB the ceilometer requirements

    This change adds ``oslo.db`` to the pip install requirements
    list which is works around an issue within ceilometer where
    the service is unconditionally importing this library even if
    its not being used.

    Closes-bug: #1624692
    Change-Id: Ib8119b39cdbce42d1d803c17612a9627daeee4dd
    Signed-off-by: Kevin Carter <email address hidden>

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

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/373392

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

Reviewed: https://review.openstack.org/373392
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=78be95bd02c8951771e919ad5f7141d7d66751d2
Submitter: Jenkins
Branch: stable/newton

commit 78be95bd02c8951771e919ad5f7141d7d66751d2
Author: Julien Danjou <email address hidden>
Date: Sun Sep 18 19:11:43 2016 +0200

    Move oslo.db to hard requirements list

    The options from oslo.db are also used by other storage drivers such as
    MongoDB, which makes it mandatory even for this backend.

    Change-Id: I3b09cffca1285ff92c0c7492b5b48c6a560251e5
    Closes-Bug: #1624692

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit fae8caee14963e1d8b031897004a39a38f0a2795
Author: Julien Danjou <email address hidden>
Date: Sun Sep 18 19:11:43 2016 +0200

    Move oslo.db to hard requirements list

    The options from oslo.db are also used by other storage drivers such as
    MongoDB, which makes it mandatory even for this backend.

    Change-Id: I3b09cffca1285ff92c0c7492b5b48c6a560251e5
    Closes-Bug: #1624692

Changed in ceilometer:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 7.0.0.0rc2

This issue was fixed in the openstack/ceilometer 7.0.0.0rc2 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 7.0.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 8.0.0

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