Conf files in $COLLECTD_CONF_DIR are loaded in alphabetical order.
If there is an error loading the python plugin, the logfile plugin is not
loaded.
To help debugging, the logfile should be loaded first.
Changing logfile.conf to 01-logfile.conf should fix this issue by making
collectd load the logfile.conf first
Closes-Bug: #1657859
Change-Id: I1f748d61d6d9a291bcc32476ac17bb0d60fdc573
pip_install collectd_ceilometer module
- Install libvirt-* requirements from the package manager so pip can build
This ensures is actually installed from teh requirements.txt file, in
relation to the related bug.
Change-Id: Iba26b1da863288e789f68646dee6a7f62fd6c160
Partial-Bug: #1644254
- Remove the pbr import in colelctd_ceiloemter/__init__.py
- PBR is still needed for a pip_install, but was causing import
errors in the plugin.
Change-Id: I24c6349caba3fdd097a5218cc048aa696bd27550
Closes-Bug: #1644254
- Create a new gnocchi dir for the collectd-gnocchi plugin.
- Get gnocchi endpoint from keystone
- Add new _base_url format
- Create metrics if they don't already exist
- Add instructions to doc/source/devstackGSG.rst
Change-Id: Id7ce8130cb22f33147b7f031cd65564375db10d6
- Change the config variable name.
- The config variable no longer references ceilometer.
- It is now clearer that the config variable is for all plugins
- The variable is a noun instead of a verb, so it's clearer that
this is something to define, instead of a boolean.
- Update docs
- Added a clarification on the format of the custom unit definition.
- Corrected incorrect reference to COLLECTD_CEILOMETER_UNITS in
devstackGSG.rst and usage.rst
Closes-Bug: #1656877
Change-Id: Ie688397242d57cf0fee6629fafb89afd8899a599
- Updated devstackGSG.rst to set CEILOMETER_BACKEND
- Added CEILOMETER_BACKEND to local.conf.minimal
- Added a reno for this bug fix
Change-Id: Ibbf606021a1f0dcc0db6c586e092b03df423a7a5
Closes-Bug: #1655611
- KEYSTONE_AUTH_URI is the admin url
- Use KEYSTONE_SERVICE_URI to access keystone as a service user.
Closes-Bug: #1656063
Change-Id: I957e166931b3dea050d0cee3365250a92ae3a57f
- Added deployment code for custom units enablement
- Updated the DevStack documentation to provide instructions on
how to enable this feature
- Editted the collectd-ceilometer-plugin.conf file to enable the
feature
- Added a reno for this change
Change-Id: Ia93da4abdf066ba677dcb3ee0e44388c06d97ff5
Add collectd-ceilometer specific contrib information
- What needs to be included in a feature?
- How to add a reno entry
Change-Id: I91cf5af30eb0e26695dffd374f796023dd8af099
- Remove installation.rst as the information has been superceeded
by devstackGSG.rst and GSG.rst.
Change-Id: Iefcb16d4a657ba591e1e20e8cff8d31a6fd47c34
Closes-Bug: #1568178
- Add -p flag when creating configuration directory
- Added a reno for this bug fix
Closes-Bug: #1648554
Change-Id: I8882738db03675a43911c04aa0172d7d3a55c5f1
- Updated the GSG.rst file with instructions on how to
enable the feature.
- Provided a reno for this documentation update.
Change-Id: I007e262f2676067ebd3bdc16dac0e26155a1f1e9
collectd-ceilometer-plugin.conf OS_TENANT_NAME was not set, therefore
was unable to connect to keystone as this value was not set during
deployment.
This change sets OS_TENANT_NAME to SERVICE_TENANT_NAME which is set by
devstack for all projects.
Change-Id: I2fdda934cb48d70bfc7f361a3916630f8aa912f7
Closes-Bug: #1649795
- Add collectd_ceilometer.ceilometer dir
- Add collectd_ceilometer.common dir
- Move test files to collectd_ceilometer/tests/{ceilometer,common}
- Add base code to collectd_ceilometer/tests
Change-Id: I5bd0a8cfe8eb55407f3d6827c0bb4b509dae8c8c
- Add collectd-ceilometer-plugin.conf to etc/collectd.conf.d/
- Add logfile.conf to etc/collectd.conf.d/
- Add default values to the collectd-ceilometer-plugin.conf
- Add default values to the logfile.conf
During deployment:
- Add substitutions for the vars
Change-Id: I196367e6f48af94c053b149596c57a735109b9ed
plugin.py is really hard to test because good part of
its logic happend during code importation.
It's test depends on the base TestCase that mocks too
much thinks in a way that makes hard to spot real problems.
The test also fails to test re-authenthication bug because
of bad mocking.
This code should fix above problems. It also remove some
logic in the hooks: exceptions are already captured and
logged by collectd and there is no need doing it inside of
hooks. Init hook is also pointless from the perspective
of this plugin initialization and it was removed.
Co-authored-by: Emma Foley <emma.l.foley@intel.com>
Related-Bug: #1615349
Change-Id: I4db8a94243ecbe98cd6bc13e4b66293172346dcd
- Added units for the following plugins: Chrony, Cpusleep, GPS,
Interface, Mysql, Ntpd, Powerdns, Protocols, Redis, Table, Xencpu
and Zfs_arc plugins.
- These are the units/plugins for collectd version 5.6.0
Change-Id: Id5499412da4b45a07ebbd333607722638224e758
in master branch keystone v2.0 api is no longer supported, this patch
is introducing v3 api support.
Change-Id: I5ed5f65f34033b6a4c550704bb186dfa8d0fc82c
Closes-Bug: #1614892
- how to prepare the system
- install,build and configure collectd
- install the ceilometer plugin for collectd
Change-Id: I5ce84fbe1243fa238dd16c164b7963f2b6f28bd4
There is code to remove collectd-ceilometer-plugin.conf when unstacking,
however, the same logic wasn't added when logfile.conf was introduced.
This change adds that logic so that logfile.conf is removed along with
collectd-ceilometer-plugin.conf
Change-Id: Ib100db5603fc22a640fe18002d754ba8b8538041
The Vagrantfile creates a basic up-to-date VirtualBox image
with Ubuntu Server 16.04 LTS and all dependencies required
to run Tox and DevStack.
Provided Vagrantfile requires Vagrant, Virtual Box. A fast
internet connection it is recommended.
If environment variables like http_proxy, https_proxy and no_proxy
it will assume a proxy connection is required to download
archives from upload repositories. In such cases
Vagrant plugin vagrant-proxyconf is required to provide proxy
enviroment to provided virtual machine.
Closes-Bug: #1606338
Change-Id: I5fc5060d78276ec5b801118d8d6873f0071ecad9
- Write unit tests for the handler
- Forward constructor parameters to the super-class
- Takes verbosity from outside instead of using configuration class
Closes-Bug: #1610348
Change-Id: I15b96d14e513889efbb50e2c51863bc224e71465
The result of POST request is threated using a more explicit
approach via raise_for_status method instead of using ambigous
bool operator.
Requests library is mocked in plugin test instead of base test class.
The test for this behaviour has been fixed.
Closes-Bug: #1614772
Change-Id: I9079246b1b501b69e850878bef13c16c7c6d0b59
Remove collectd_ceilometer.tests.base.TestCase from base classes
of collectd_ceilometer.tests.base.test_config test cases.
Related-Bug: #1615349
Change-Id: Ic407752a07fb96a5fc431e4253e976148800b5ea
When creating the collectd-ceilometer-plugin.conf file,
incorrect values were used
This commit changes the following variables:
- OS_USERNAME "$OS_USERNAME" -> OS_USERNAME "ceilometer"
the service username is hardcoded, and OS_USERNAME overrides an
existing variable used by OpenStack
- OS_PASSWORD "$OS_PASSWORD" -> OS_PASSWORD "$SERVICE_PASSWORD"
these are usually identical for devstack, but can be set differently
- OS_TENANT_NAME "service" -> OS_TENANT_NAME "$SERVICE_TENANT_NAME"
set to service by default.
This change also removes the code that overwrote the OS_USERNAME
and OS_PASSWORD variables in devstack/settings
Change-Id: I9c8435766aa8448210bf06676f4e06d8fbf3a7a8
Closes-Bug: #1605352
