Dependency of gnocchiclient in ceilometer

Bug #1621384 reported by Gautam Prasad
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Mehdi Abaakouk

Bug Description

Ceilometer throws the following error:

File "/usr/lib/python2.7/site-packages/ceilometer/cmd/storage.py", line 24, in <module>
from ceilometer import gnocchi_client
File "/usr/lib/python2.7/site-packages/ceilometer/gnocchi_client.py", line 14, in <module>
from gnocchiclient import client
ImportError: No module named gnocchiclient

gnocchiclient is not listed as dependency in requirements.txt.

The broader question is why gnocchiclient should be a dependency when gnocchi is not used in plain ceilometer installation?

Revision history for this message
gordon chung (chungg) wrote :
Changed in ceilometer:
status: New → Triaged
importance: Undecided → Low
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/369895

Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ceilometer (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/369896

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

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

commit 18ad1c15192943cc8dab3867c54d59cbbfca628e
Author: Mehdi Abaakouk <email address hidden>
Date: Wed Sep 14 09:43:21 2016 +0200

    Don't require gnocchiclient

    Closes-Bug: 1621384
    Change-Id: I0480f2dd5e022c1abe210325eb86b5e51582c18d

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

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

commit 3e013ff8f4b84f524f62152d293321cecbc1856b
Author: Mehdi Abaakouk <email address hidden>
Date: Wed Sep 14 09:50:39 2016 +0200

    Use deps extra for optional requirements

    Related-Bug: #1621384
    Change-Id: I227c121b69ac1f51ed922e0b11bec005928aacab

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.