ceilometer devstack plugin not efficient in multinode scenario

Bug #1508518 reported by Chris Dent
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
ZhiQiang Fan

Bug Description

If you want to run ceilometer via a multinode devstack setup the devstack/plugin.sh code is not well optimized to handle this: services and storage which are not required on e.g. a compute node are started. Some examples:

* if CEILOMETER_USE_MOD_WSGI evaluates to true, the api service will be started even if "disable_service ceilometer-api" has happened
* if ceilometer-collector is not enabled, the database is still created on the current node

There are probably others, but that's a start.

gordon chung (chungg)
Changed in ceilometer:
importance: Undecided → Medium
status: New → Triaged
ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
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/238583

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

for database issue, we need to discuss how complicated we want to support,

for a simple 1 controller + X compute model, we can install according to collector service

for multiple controllers, we need to specify which node want to install database service, but I think we should not support such model for now, because I have no use case for such scenario, devstack is not a good choice for too complicated scenario I think, we have openstack-puppet and openstack-ansible for it

what's your opinion?

Revision history for this message
Chris Dent (cdent) wrote :

I think the right thing is keep it simple: if collector is enabled for this node, install the database, otherwise don't. While that doesn't cover all the available scenarios it does get the common ones while correcting the presenting problem (installing the database on compute only nodes).

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

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

commit 9c3278c942e07cc0f94439895cffafbe6697afbc
Author: ZhiQiang Fan <email address hidden>
Date: Thu Oct 22 08:37:14 2015 -0600

    configure Apache only when ceilometer-api is enabled

    Currently, even though ceilometer-api service is disabled by user,
    we still try to configure ceilometer-api in Apache if CEILOMETER_USE_MOD_WSGI
    is not set to false. This patch adds a constraint to ensure we only
    configure when ceilometer-api is enabled.

    Change-Id: I3f2bab3f646f7df57c32db3251f811cb801d93de
    Partial-Bug: #1508518

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

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

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

commit 0ba0ba7270973c8a9558f863428ecefe120c9467
Author: ZhiQiang Fan <email address hidden>
Date: Wed Nov 11 01:15:58 2015 -0700

    install database when collector is enabled.

    Currently, if we run devstack for compute node, ceilometer-collector
    service will be disabled on compute node, but the database is still
    installed.

    This patch makes sure that database is installed only when collector
    is enabled.

    Also add check for coordinate and hypervisor requirements.

    Change-Id: Ib3614d2e698403c24ad299d95ef42c81815fa76e
    Closes-Bug: #1508518

Changed in ceilometer:
status: In Progress → Fix Committed
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
Liusheng (liusheng)
Changed in ceilometer:
milestone: none → mitaka-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.