Cannot disable ceilometer::collector without having to include ceilometer::db

Bug #1352958 reported by Mathieu Gagné
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceilometer
Fix Released
Undecided
Mathieu Gagné
Icehouse
Fix Released
Undecided
Mathieu Gagné

Bug Description

You cannot disable ceilometer-collector without including ceilometer::db

  class { '::ceilometer::collector':
    enabled => false
  }

If ceilometer::db is not included, catalog compilation will fail with this error:

  Error: Could not find dependency Class[Ceilometer::Db] for Service[ceilometer-collector]

You shouldn't have to setup database configuration to disable a service.

Changed in puppet-ceilometer:
assignee: nobody → Mathieu Gagné (mgagne)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceilometer (master)

Reviewed: https://review.openstack.org/112054
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceilometer/commit/?id=8c9f4fcd53e4ef0fb7694de6dce6e001c7e8cf7d
Submitter: Jenkins
Branch: master

commit 8c9f4fcd53e4ef0fb7694de6dce6e001c7e8cf7d
Author: Mathieu Gagné <email address hidden>
Date: Tue Aug 5 11:58:27 2014 -0400

    Add ability to disable ceilometer-collector without db

    You are no longer required to include ceilometer::db when
    disabling ceilometer-collector using the enabled parameter.

    The relationships with any database resources are now only enforced
    when the service is enabled.

    Closes-bug: #1352958
    Change-Id: I00a5e651184788acdd30de1908e2bf38566f78e1

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

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/112356

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

Reviewed: https://review.openstack.org/112356
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceilometer/commit/?id=43ad68554622888d593c9b08c54f09dd962fc5c9
Submitter: Jenkins
Branch: stable/icehouse

commit 43ad68554622888d593c9b08c54f09dd962fc5c9
Author: Mathieu Gagné <email address hidden>
Date: Tue Aug 5 11:58:27 2014 -0400

    Add ability to disable ceilometer-collector without db

    You are no longer required to include ceilometer::db when
    disabling ceilometer-collector using the enabled parameter.

    The relationships with any database resources are now only enforced
    when the service is enabled.

    Closes-bug: #1352958
    Change-Id: I00a5e651184788acdd30de1908e2bf38566f78e1
    (cherry picked from commit 8c9f4fcd53e4ef0fb7694de6dce6e001c7e8cf7d)

Mathieu Gagné (mgagne)
Changed in puppet-ceilometer:
milestone: none → 5.0.0
status: Fix Committed → Fix Released
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.