Commit Graph

186 Commits

Author SHA1 Message Date
OpenDev Sysadmins cdf9ff6978 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:50:30 +00:00
Monty Taylor 55db3ffc26 Update .gitreview for new repo name
Change-Id: I733621a2edbf481f5c0b4a81addeb5c9110cf41c
(cherry picked from commit ed862ab226)
2018-11-18 12:20:26 +00:00
Emma Foley 1defcfb65c Add DEFAULT_ARCHIVE_POLICY config option for gnocchi
Added a DEFAULT_ARCHIVE_POLICY option to Gnocchi plugin,
The user can now configure the archive policy that the metrics
should use.

Change-Id: I445c4bc98b6431a39cf805a00b6237d8090a0b98
2018-03-14 19:00:16 +00:00
Emma Foley 6ecac9f099 remove libvirt from requirements.txt
The libvirt installation via pip was failing, so this fixes the gate change too.
libvirt-python is installed via the system package manager during deployment. It does not need to be
reinstalled via pip.

Change-Id: I9d25ef6c892b10d56c05279bd0d0633c9e38ac4f
2018-03-14 18:35:24 +00:00
Emma Foley 40de219964 gnocchi: Remove archive-policy when creating metrics
Archive policy rules are created by the operator, and assign
archive policies to metrics at creation based on certain rules,
if there isn't an archive policy explicitly assigned to the
metric already.

This patch removes the explicit archive policy parameter when
creating the metrics, this means that the archive policy rules
defined by the oparator are respected.

Change-Id: Ie08df4e817fc701e4b524cdaaf181570fcc541f5
Related-Change: I562f62e1236c7c01c79a5e465ebec103c7bb07bd
2017-10-20 12:37:35 +00:00
Emma Foley ead2004b80 common/sender: Deal with ReadTimeout in send()
If there's a ReadTimeout when sending a HTTP request, the
plugin now retries before exiting.

Change-Id: I0218bd412419079ec9544810a6ab863f3de1a915
(cherry picked from commit 02b65dc416)
2017-09-28 12:46:29 +00:00
Jenkins f004cdc6d2 Merge "[refactor] common/test_logger.py: move common code to setUp()" 2017-09-26 09:41:40 +00:00
Jenkins 962eb9dd76 Merge "[refactor] Move tests from aodh/test_plugin to aodh/test_sender" 2017-09-26 09:32:58 +00:00
Jenkins 2c15be9f91 Merge "Remove sample response from common/keystone_light There is a sample keystone response at the end of common/keystone_light that is not needed." 2017-09-26 09:32:52 +00:00
Emma Foley f6fd19b620 [refactor] common/test_logger.py: move common code to setUp()
* Add setUp()
** Add self.handler, self.collectd, self.config, self.logger members
* remove __init__()

Change-Id: Ic8e52f34fc4a479f34bf714bba93008a3c8f3d25
Partial-Bug: #1714012
2017-09-26 09:29:14 +00:00
rajat29 a336ebfea5 Remove sample response from common/keystone_light
There is a sample keystone response at the end of
common/keystone_light that is not needed.

Closes-Bug: #1718670

Change-Id: Ieba434d0bc9ee3c47eeb4464613bfbf04cc38b75
2017-09-22 10:50:04 +05:30
Jenkins 0a701d3fb9 Merge "[refactor] Add dummy test_{sender,writer}.py files" 2017-09-21 10:43:59 +00:00
Emma Foley c230403425 [refactor] Add dummy test_{sender,writer}.py files
* Tests for gnocchi should be written into the appropriate
  test_{sender,writer}.py files

Change-Id: Ia11e227d6e11ccc1f63d7d9a364b7c01d4c25e67
2017-09-21 10:32:16 +00:00
Emma Foley e4c6469429 [refactor] common/test_config: Remove duplicated config declaration in tests
* Added setUp() call
* Moved declaration of config to setUp()
* config -> self.config

Change-Id: I07a663552c5b6333630f381961411acd8ec39f73
Partial-Bug: #1714012
2017-09-19 16:35:34 +01:00
Emma Foley 1e790c13ad [refactor] Move tests from aodh/test_plugin to aodh/test_sender
A number of tests for aodh.sender.Sender are in
aodh/test_plugin.py. Since aodh/test_sender.py was recently
added, it makes sense to move the sender tests to this new
location.

The following tests were moved:
* test_get_alarm_state_severity_low
* test_get_alarm_state_severity_moderate
* test_get_alarm_state_severity_critical

The tests were refactored to use self.sender that is defined
in TestSender/setUp().

Change-Id: I337c37b14f593183a46b8fd4b077c5b47bc791bc
2017-09-13 21:09:37 +00:00
Jenkins 46ea48fb44 Merge "common/test_sender: Update test for perform_request" 2017-09-07 11:12:41 +00:00
Jenkins 38ab811ebd Merge "Add release note for metername update" 2017-09-07 11:08:27 +00:00
Emma Foley a93c57e61f common/test_sender: Update test for perform_request
* Add test_perform_request_req_type_put
* Updated test_perform_request_req_type_post
  * Now tests the default case

Change-Id: Iff7bc4d4370c5517e481db15ae1318f2e4fd34a5
2017-09-06 15:28:02 +00:00
Emma Foley 66d495696f Removes local.conf used by vagrant
The "Removing Vagrant support" commit missed the local.conf
used to deploy devstack within the Vagrant VM.
It is removed in this commit.

Related-Bug: #1710856
Change-Id: Ie3f4533f91ad13efabe481abd658323bd19a5277
2017-09-06 15:23:26 +00:00
Emma Foley ebe287370a Add release note for metername update
Change-Id: Iceb0bee94243bc02693bd250bd20ba846a6c8f22
2017-09-06 08:50:18 +00:00
Jenkins d1ce958bee Merge "Updated _get_alarm_id function in aodh plugin" 2017-09-04 15:35:32 +00:00
Jenkins 7089299f8c Merge "Removing vagrant support" 2017-08-29 15:05:51 +00:00
Ivan Dyukov 680a43d179 Updated _get_alarm_id function in aodh plugin
- Request server for alarm_id before creating new one
  - Adding http get method to _perform_request of common Sender
  - Add unit tests for {aodh,common}/sender

Change-Id: I48e83e87349048962c400a2660cd0c0252563ea0
Closes-bug: #1672296
Signed-off-by: Ivan Dyukov <i.dyukov@samsung.com>
2017-08-29 16:21:19 +03:00
rajat29 eef5c8dce0 Inconsistent capitalization on Gnocchi, Aodh, OpenStack in docs
Docs use inconsistent capitalisation of Gnocchi, Aodh, OpenStack, etc.
Ref: https://docs.openstack.org/contributor-guide/writing-style/openstack-components.html

Change-Id: Ia1ea04d4cf45e33f0257226e09e5a4da5275c4ad
Closes-Bug:#1708667
2017-08-29 03:51:49 +00:00
Mark O'Neill ef5ca1ab76 Removing vagrant support
Vagrant support is untested and unused.
Closes-Bug: #1710856

Change-Id: Iac795b092f25c0a0d062589ae965921cbdc0ab99
2017-08-28 16:19:38 +01:00
Jenkins d474b8ba68 Merge "Fix inconsistencies in usage.rst" 2017-08-17 17:09:36 +00:00
Jenkins 0ef1e1bda6 Merge "Multi-node deployment configuration" 2017-08-17 16:53:28 +00:00
Jenkins e17da5ec73 Merge "Modifies metername and resource_id" 2017-08-16 14:28:59 +00:00
Helena McGough 07ae3f81ba Multi-node deployment configuration
- Automated the required configurations for the collectd.conf file
 - Enables the virt and network plugins
 - Included a MULTI_NODE_DEPLOYMENT configuration option for this change
 - This is only enabled when the libvirt.py file is not in use
 - Provided instructions on how to deploy multi-node

Change-Id: I151e318074250b94dbef247e6858c78086fcd362
2017-08-16 12:12:13 +01:00
Emma Foley 5717989110 Modifies metername and resource_id
* Changes metername value:
   <plugin>.<type> -> <type>.<type_instance>
   e.g. cpu.cpu -> cpu.nice
* Changes resource_id value
   <hostname>-<plugin_instance>-<type_instance> ->
   <hostname>-<plugin>-<plugin_instance>
   e.g. localhost-0-nice -> localhost-cpu-0
* Changes metername in gnocchi/writer
    metername -> <resource_id>@<metername>

* Supports multi-host deployment

Change-Id: If4cbff1c624ceb1e46b86bf999559b159ee78a04
2017-08-16 10:19:06 +00:00
Hangdong Zhang fde7ba8bfb Update URLs in documentation
Update URLs according to OpenStack document migration
BTW: Do some optimization as well (http -> https)

Change-Id: Ie9cc2c831186cd4dbbde08973032aece20540fc2
2017-08-16 09:34:37 +08:00
Emma Foley 8b016ca204 Fix inconsistencies in usage.rst
- Config options need to be headings
- 1 blank line between description and default
- 2 blank lines between each entry
- Order config options alphabetically

Change-Id: I9d9d73c15e73fe282ddc87e17ff19693a46db831
2017-08-15 17:42:46 +00:00
Elizabeth Burke d256ab054a Deleting ceilometer from the repository
-remove Collectd_Ceilometer folder
-remove Ceilometer specific unit tests
-Remove mentions in devstack, doc, and etc folders
-Substitute refereces of Ceilometer with Gnocchi/Aodh

Closes-Bug: #1710855
Change-Id: I210ca8182c489c4f90a6521b1b7fd6f8a391e602
Signed-off-by: Elizabeth Burke <elizabeth.burke@intel.com>
2017-08-15 16:00:14 +00:00
Emma Foley f86c539aaf Add space between parameters
Change-Id: I239eb0a4b6f5e689e68a71f5880ca46538019550
2017-08-14 19:28:20 +00:00
Jenkins 69dbbcfb29 Merge "Add unit mappings for new plugins collectd 5.7.2" 2017-08-14 15:32:06 +00:00
Jenkins 625aa325ec Merge "Fix plugin logging" 2017-08-14 15:20:14 +00:00
Helena McGough 7ef22179fa Deprecated the libvirt meter
- Libvirt meter is only enabled if configured
 - Added deployment code
 - Included a reno
 - Updated doc/source/usage.rst
 - Updated unit tests

Change-Id: Ia4eb566e087ff341505896534296e7afd338f98b
2017-08-14 11:38:33 +00:00
Mark O'Neill 99bf568763 Add unit mappings for new plugins collectd 5.7.2
Unit mappings for dpdkstat, hugepages, intel_rdt

Change-Id: Ia06d61493a2a0600684520960f5cf5b9297e0b70
2017-08-01 16:03:47 +00:00
Jenkins b24bb7cee8 Merge "Update docs to include Aodh and Gnocchi" 2017-07-26 16:26:00 +00:00
Jaroslav Safka f903de3f10 Fix plugin logging
Fix logging of debug, info and warning levels.
Also configuration parameter VERBOSE is working again.

fixes problems:
* VERBOSE parameter can be changed at runtime
* Initial log level was unset, then there was nothing logged lower than
exception level. (for example configuration dump was not logged)
* Initial VERBOSE was set to True, then it was too much verbose. And
this flag was mean to be used for debugging problems.

NOTE: real log level is driven by collectd configuration, not by plugin
config.

Change-Id: Ia7048ccb74f27a5d5885b9c0bda17d6fba603e9b
Closes-Bug: #1664973
2017-07-19 09:32:41 +00:00
Jenkins 0aa39651ee Merge "Included the aodh sender class in the common one" 2017-07-17 01:58:11 +00:00
Jenkins 19fe922da4 Merge "Make alarm severity configurable" 2017-07-17 01:55:18 +00:00
Jenkins 73792e84fb Merge "Move SampleContainer to collectd_ceilometer/common" 2017-07-14 13:47:44 +00:00
Emma Foley e63e7cafab Update docs to include Aodh and Gnocchi
README.rst:
* Updated introductory paragraph to include gnocchi and aodh
* Updated features list to include gnocchi and aodh features

doc/source/GSG.rst:
* Updates collectd installation instructions
* Updates intro to configuring collectd as a service
* Updates packages to be installed
* Updates installation and configuration for plugins
* Updates verification steps for ceilometer, aodh and gnocchi

Change-Id: I43698ea898bb990f163efc938ef6bbbec1e16b58
2017-07-13 17:36:26 +00:00
Helena McGough 2223d14005 Included the aodh sender class in the common one
- Combined commonalities from the aodh and common sender classes
 - Updated all of the relevant tests for each class
 - Included a reno

Change-Id: I78fcdb075661e125b0e9f16bdee626cd811ad78e
Closes-bug: #1668210
2017-06-30 12:41:49 +00:00
Helena McGough 1437e06bf1 Make alarm severity configurable
- Enabled the user to configure the severity of alarms triggered
   by metrics/meters
 - Provided/Updated documentation for this changea
 - Included all of the necessary tests for this change
 - Included a reno for this change

Change-Id: Icefe2edf7695c4a8b3d5838844e2514b9876967f
2017-06-30 10:04:57 +00:00
Helena McGough 4180fb5a76 Updated _get_alarm_state function
- Get the state of an alarm based on the severity of the collectd
   notification that is sent and not the message.
 - Added a bug fix reno for this.
 - Reduced the probability that the state will return "insufficient
   data".

Change-Id: Ic993337aa49aed1d189708f5d8cd3935e6d1bbe1
Closes-bug: #1681392
Closes-bug: #1672301
2017-06-26 17:46:37 +00:00
Emma Foley a3effa6023 Move SampleContainer to collectd_ceilometer/common
SampleContainer was duplicated in ceilometer/ and gnocchi/

- Moved SampleContainer to collectd_ceilometer/common
- Added unit tests for SampleContainer

Change-Id: Id394c6b489d69840ff5de1eb07f1a4b357d3bb22
2017-06-23 14:24:17 +00:00
Helena McGough e2ce9e8ffd Added test for _update_alarm
- Provided _update_alarm and alarm_not_updated test.

Change-Id: I476fcb3503357b4dfac5ef741494032cc0fc6090
Closes-Bug: #1669814
2017-05-26 09:03:47 +00:00
Helena McGough 3b88d50820 Refactor update and create alarm functions
- Checks for errors and retries creating/updating alarms which fixes
    the bug, by removing the need to have a dictionary for alarms.
  - Included a get_alarm_id function.
  - Refactored the update and create alarm functions.
  - Updated and included the relevant tests for this.
  - Include a reno for the bug fix.

Change-Id: I7e998bc8cb55f9ef7564464fcd4eded06643e1eb
Closes-Bug: #1677597
2017-05-26 08:37:27 +00:00