Merge remote-tracking branch 'github/pr/2467'
clang-format -style=file -i src/daemon/{collectd,configfile,globals}.c
Minor changes to address PR comments.
Tidy up formatting with clang-format.
Merge remote-tracking branch 'github/pr/2490'
rrdtool plugin: Small code cleanup.
Primarily to test the clang-format bot.
Primarily to test the clang-format bot.
Merge remote-tracking branch 'github/pr/2488'
clang-format -style=file -i src/snmp.c
Set timeout initially to 0 per reviewer's suggestion
Adress reviewer's suggestions and add documentation
Merge remote-tracking branch 'github/pr/2489'
Merge pull request #2485 from rpv-tomsk/processes
processes: Improved documentation
processes: Improved documentation
processes: Improved documentation
snmp plugin: Use cf_util_get_boolean() for boolean config options.
filecount: Controls whether or not to include only regular files in the count (#2483)
Address reviewer's suggestions
Add timeout and retry config options for SNMP plugin
Merge pull request #2482 from outini/master
Implementation of "Prefix" option for write_http plugin
Implementation of "Prefix" option for write_http plugin
write_http: Allow to set empty metrics prefix in KAIROSDB format
write_http: Added strdup() result check
Setting metrics_prefix as char const *
Added denis pompilio to contributors
Implementation of "Prefix" for write_http plugin
This option is only available for the KAIROSDB format
This option defaults to "collectd"
This option is only available for the KAIROSDB format
This option defaults to "collectd"
Leftover reference to hostname_g[].
Make g_strings in perl.c not global so that hostname_g as a char* doesn't break it.x
Merge pull request #2300 from rpv-tomsk/fix-sread
Remove close() call from sread()
Remove close() call from sread()
Merge remote-tracking branch 'github/pr/2135'
python: Addressed review comments
- Run clang-format
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
- Run clang-format
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
Changed return type of sread() and swrite() from 'ssize_t' to 'int'
Functions 'sread()' and 'swrite()', unlike 'read()' and 'write()', does not return the number of bytes actually read/written.
They return error status, which is returned as 'int' type in Collectd code.
Functions 'sread()' and 'swrite()', unlike 'read()' and 'write()', does not return the number of bytes actually read/written.
They return error status, which is returned as 'int' type in Collectd code.
Proper check for `swrite()` return value
The `swrite()` can return positive value from `errno` in case of error.
The `swrite()` can return positive value from `errno` in case of error.
Remove close() call from sread()
The sread() function should not close fd in any cases, that is unexpected.
The sread() function should not close fd in any cases, that is unexpected.
Merge pull request #2024 from rpv-tomsk/master-ipmi
ipmi plugin: Implemented IPMI LAN connection and plugin improved
ipmi plugin: Implemented IPMI LAN connection and plugin improved
ipmi plugin: style
ipmi plugin: Refactored to use c_ipmi_notification_init_()
ipmi plugin: Fixed remarks found while review
memcached: Fix hitratio calculation
If there is no requests, NAN is reported as hitratio.
If there is no hits - zero is reported.
If there is no requests, NAN is reported as hitratio.
If there is no hits - zero is reported.
ipmi plugin: Use cf_util_get_boolean()
memcached: style changes
memcached: Replace gauge_t with derive_t
memcached: Renamed calculate_rate to calculate_ratio_percent
memcached: Fix hitratio reporting
When Collectd calculates 'hitratio', it divides two continiously-grown values of Memcached stats.
As result, reported metric contains the average since Memcached start, which is incorrect.
When Collectd calculates 'hitratio', it divides two continiously-grown values of Memcached stats.
As result, reported metric contains the average since Memcached start, which is incorrect.
memcached: Fix CPU usage reporting
ipmi plugin: Added a define for errors / messages buffer size
ipmi plugin: Notify about IPMI connection state change
ipmi plugin: Improvements
* Do not request reading unless domain is connected
* Report instance name in error messages
* Do not request reading unless domain is connected
* Report instance name in error messages
style
ipmi plugin: Code cleanup
Currently curl_json will barely ignore boolean values in a non erroneous
way, so that if you have an array like this:
you will be able to access the 123 number like
We now rather call our number parser with 0 for false, 1 for true.
way, so that if you have an array like this:
you will be able to access the 123 number like
We now rather call our number parser with 0 for false, 1 for true.
ipmi plugin: Implemented IPMI LAN connection and plugin improved
* Added instances support
* Removed the sensor removal in case of reading errors
* Added the lock to do not request new readings until previous reading is complete
* Disabled SDRs local cache
* Added the check for sensors 'is readable' flag
* Added the check for sensor event reading type (if sensor is discrete or not)
* Added checks for sensor states 'scanning disabled' and 'initial update in progress'
* Changes in error reporting
* Added instances support
* Removed the sensor removal in case of reading errors
* Added the lock to do not request new readings until previous reading is complete
* Disabled SDRs local cache
* Added the check for sensors 'is readable' flag
* Added the check for sensor event reading type (if sensor is discrete or not)
* Added checks for sensor states 'scanning disabled' and 'initial update in progress'
* Changes in error reporting
Fixed issues with initializing hostname.
Fix kstat_ctl_t definition in globals.
Cleanup some defines and includes.
Adds setter for hostname_g and eliminates hostname_g_size variable.
Add MIT license to new globals files.
Merge remote-tracking branch 'github/pr/2469'
Merge remote-tracking branch 'github/pr/2453'
Merge remote-tracking branch 'github/pr/2398'
libcollectdclient/network_parse.c: Ensure memory allocated in parse_values() is always freed.
Shoutout to scan-build for finding this one!
Shoutout to scan-build for finding this one!
Merge remote-tracking branch 'github/pr/2091'
ipmi: Run code through clang-format
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
ipmi: Addressed review comments
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
Signed-off-by: Mytnyk, Volodymyr <volodymyrx.mytnyk@intel.com>
Merge branch 'collectd-5.7'
Merge branch 'collectd-5.6' into collectd-5.7
Bump version to 5.6.3; Update ChangeLog.
Reading configuration file is now done outside main().
Common stuff now builds as a library.
Merge remote-tracking branch 'github/pr/2466' into collectd-5.7
intel_pmu: address PR comments
Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>
Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>
write_prometheus plugin: Don't construct struct sockaddr_in6 manually.
On FreeBSD, the struct is an incomplete type.
On FreeBSD, the struct is an incomplete type.
write_prometheus plugin: Add support for libmicrohttpd < 0.9.0.
0.9.0 was released in July 2010, yet Precise, which we're still building for,
ships 0.4.6. *sigh*
0.9.0 was released in July 2010, yet Precise, which we're still building for,
ships 0.4.6. *sigh*
write_prometheus plugin: Fall back to IPv4 socket if IPv6 fails.
Unfortunately, for libmicrohttpd "dual stack" means "enforce IPv6 (but
allow IPv4, too)". That means that on systems without IPv6,
libmicrohttpd'd initialization fails (when the dual stack flag is set).
To work around this limitation we open the socket ourselves and pass it
to libmicrohttpd to use.
Fixes: #2149
Unfortunately, for libmicrohttpd "dual stack" means "enforce IPv6 (but
allow IPv4, too)". That means that on systems without IPv6,
libmicrohttpd'd initialization fails (when the dual stack flag is set).
To work around this limitation we open the socket ourselves and pass it
to libmicrohttpd to use.
Fixes: #2149
intel_pmu: refactor pmu_event_get_meta function.
Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>
Signed-off-by: Serhiy Pshyk <serhiyx.pshyk@intel.com>
ceph plugin: Fix comment typo.
Merge remote-tracking branch 'github/pr/1558'
curl_xml: Use of xmlFree() for values returned by xmlNodeGetContent()
curl_xml: Fixed issues found by review
varnish plugin: Metrics with invalid data source type. #2462
curl_xml: Updated documentation and default configuration
curl_xml: Code cleanup and fixes
* XPath list creation moved from cx_config_add_xpath() to cx_config_add_url()
* Added missing check for ValuesFrom option presence
* Added missing check for strdup() result
* Removed excessive check for option name in cx_config_add_url()
* XPath list creation moved from cx_config_add_xpath() to cx_config_add_url()
* Added missing check for ValuesFrom option presence
* Added missing check for strdup() result
* Removed excessive check for option name in cx_config_add_url()
collectd.spec: bzip2 is the default
curl_xml: Removed excessive operation
curl_xml: Refactored via cx_get_text_node_value()
curl_xml: Code cleanup, clang-format
curl_xml: Renamed 'PluginName' option to unified 'Plugin'.
curl_xml: Changed documentation to match proposed changes.
curl_xml: Add ability to set plugin instance from XML data.
curl_xml: Add ability to set custom plugin name in collected data.
Updating the varnish plugin to use varnish-plus counters (mainly MSE)
Merge branch 'pr/2454'
processes plugin: Format with clang-format.
Also a small manpage formatting tweak.
Also a small manpage formatting tweak.
Merge branch 'collectd-5.7'
Merge branch 'collectd-5.6' into collectd-5.7
use existing type to publish the data
there no such command in make make-dist-bz2
collectd.conf(5): Move towards unifying "string literals".
Merge remote-tracking branch 'github/pr/1707'
dbi plugin: Fix additional memory leaks.
dbi plugin: Fix memory leak.
Some memory was leaked when shutting down.
This is a partial back-port of 5ee365a.
Signed-off-by: Florian Forster <octo@collectd.org>
Some memory was leaked when shutting down.
This is a partial back-port of 5ee365a.
Signed-off-by: Florian Forster <octo@collectd.org>
collectd.conf(5): Move towards unifying "string literals".
Merge remote-tracking branch 'github/pr/1681'