collectd.conf file not being updated correctly

[Maryam Tahhan]

Hi
i've been trying to stack the stable/newton branch and I noticed this snippet is missing from collectd.conf

<Include "/opt/collectd/etc/collectd.conf.d">
        Filter "*.conf"
</Include>

Note that I built collectd separately. but I would still think devstack would manage this step for my.

# Enable Collectd
enable_plugin collectd-ceilometer-plugin https://git.openstack.org/openstack/collectd-ceilometer-plugin stable/newton
#enable_plugin collectd_ceilometer_plugin /opt/stack/collectd-ceilometer-plugin
COLLECTD_INSTALL=false
COLLECTD_CONF_DIR="/opt/collectd/etc/collectd.conf.d"

note when I install collectd manually the conf file collectd.conf lives in "/etc/collectd/"

BR
Maryam

[Emma Foley (emma-l-foley)]

When installing from package, this snippit is already there.

Devstack treats a custom build the same as it treats installation from packages, so it doesn't modify any conf files for the service.

It should be a trivial change to implement this, as long as the COLLECTD_CONF_DIR is set.

    At the installation step, check the usual places for collectd.conf (COLLECTD_CONF_DIR/../collectd.conf), and see if it contains the include block.

However, this isn't high priority right now, and might become irrelevant when the code to build from source in merged.