Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added HTTP protocol #6

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Added HTTP protocol #6

wants to merge 37 commits into from

Conversation

jsabin
Copy link

@jsabin jsabin commented Nov 17, 2014

Added HTTP Protocol.
Added property (MetricName) to customize the metric name.
Removed UDP connection since KairosDB does not support UDP.
Added documentation for the properties in README.md.

This change is not backward compatible for a couple of reasons:

  1. Renamed "TCP" to "TELNET" since both Telnet and HTTP are TCP protocols.
  2. Removed HostPostfix and MetricPrefix properties and added MetricName property instead.

Jeff Sabin and others added 30 commits November 12, 2014 17:06
…nd MetricPrefix properties. Host tag is now added by default. Added AddHostTag to turn off adding host tag if desired.
… bug where the lowercase_metric_names property was not being used. Removed UDP socket connection since KairosDB does not communicate over UDP. Added property defaults to the readme.md file.
…tocol parameters. Also fixed param validation.
… the current directory for testing rather than relying on it to exist in the collectd installation directory.
…that the key for the counters map was not built correctly but was using the naming convention passed in from config. The second was that if a type consisted of multiple values, only the last value was sent to kairos.
…y to pick up plugins you may not want. Fixed bug where values with multiple types where some of the types are not counters were not sent.
…in formatters are not defined for a given plugin.
…emoves the need for global variables. Added PluginFormatterPath property. This is the directory where plugin formatters are loaded from. Created example plugin formatters. Changed the name of the "format" method in the formatter because format is a built in function for Python. This avoids confusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants