tokkee.org

plugin.h: Introduce the identifier_t type with pointer fields.

This serves two purposes:
 - Allow field values of arbitrary length.
 - Reuse the identifier in other places.

Renamed identifier_t from utils_vl_lookup to lookup_identifier_t.

command parser: Add a vector-based interface.

The new interface expects a parsed list of fields. Parsing of input buffers
is now done centrally in the generic code and the existing parser is a
wrapper around this and the new vector-based parser. The actual command
parsers (PUTVAL for now) receive the parsed list of fields.

As a side-effect, this changes the parser behavior a little:

 - "foo=a string with spaces or \\ special characters" is now valid syntax
   for specifying options in addition to foo="value with spaces, etc.".
 - foo= is now a valid option even without quotes around the empty value.

Add a generic interface for parsing the text protocol.

Implement it for PUTVAL for now.

The text protocol is used in multiple places and the parser will avoid code
duplication in client programs which can, instead, use it to generate the
respective requests.

Use the 'cmd_' prefix for public functions related to command handling.

README: Document gRPC's client (sender) support.

gRPC plugin: Add Florian to copyright holders.

Merge remote-tracking branch 'github/pr/631'

Merge branch 'pr/1649'

dpkgstat plugin: Filter through clang-format.

dpdkstat: Fixed issue with incorrect argc calculation

Changed log messages to use PRIuN defines

Change-Id: I1433fbd6443c3d0a956c04670f93615042b40577
Signed-off-by: Taras Chornyi <tarasx.chornyi@intel.com>

Merge remote-tracking branch 'github/pr/1942'

Merge remote-tracking branch 'github/pr/1939'

Update email address

hddtemp: Improve robustness of test

Don't rely on signedness of buffer_size.

hddtemp: Lift 1024-byte response limit

Dynamically scale the response buffer, up to a maximum of 1 MB.

hddtemp: Lift 32-device limit

Instead of building an array of fields and then walking it, split into
fields as we go.

dpdkstat: Addressed PR comments

1) Added missing option description
2) Replaced strerror with sstrerror
3) Do not return from helper code
4) Use generic collectd Interval implementation
5) Removed "\n" from log messages

Signed-off-by: Taras Chornyi <tarasx.chornyi@intel.com>

parse_value_file: adding strstripnewline back

use fgets in parse_value_file, fixes #1941

src/daemon/utils_time.c: Fix invalid strcpy position in format_rfc3339().

Plug a leak in Target:Set.

Merge branch 'pr/1918'

src/daemon/utils_time.c: Pass "struct tm *" to format_zone().

This is a partial revert of e2cb258c7b6ce456f4119fd1454c85b479fa3e2d:
strptime() does not look the local timezone up itself but gets the
information from the "struct tm". If that is initialized with {0}, it
will always return the "+0000" time zone.

perl plugin: Renamed and documented "RegisterLegacyFlush" option.

perl plugin: Added 'flush' fallback

perl plugin: Changed pluginname form to allow single perl plugin flush. (Part 2, fixed _plugin_unregister_generic() function to match _plugin_register_generic_userdata().)

perl plugin: Fixed Collectd::plugin_call_all function name im DEBUG()/ERROR()

perl plugin: Mark thread as running to avoid deadlock moved from perl_shutdown() to c_ithread_destroy()

perl plugin: Changed pluginname form to allow single perl plugin flush.

perl plugin: Register perl plugins with use of 'userdata'.

Address more review comments:
- Repurpose rfc3339()/rfc3339nano() to return time in zulu format.
- Add formatted time examples to function comments.
- Clean up helper functions and add variable iniitalization.

Force use of local time in the postgresql plugin.
Update postgresql docs to reflect reality.

Address review comments:
- Repurpose rfc3339/rfc3339nano to use UTC.
- Add rfc3339_local/rfc3339nano_local for local time.
- Factor out common bits; saner helper functions.
- Update comments.

Merge branch 'pr/1792'

src/daemon/common.[ch]: Reimplement strjoin().

This new implementation truncates fields rather than aborting when there
is more space in the output buffer. Since strjoin() is mostly used to
fill plugin and type instances, which are otherwise usually filled with
sstrncpy(), i.e. also truncate the string rather than erroring out.

The unit test has also been rewritten to test the new functionality.

The new functions have been formatted with clang-format.

Fixes: #1792

hugepages plugin: Add "g_" prefix to global variables.

hugepages plugin: Add myself to the list of authors.

hugepages plugin: Remove an unused variable.

hugepages plugin: Implement the "ValuesPages", "ValuesBytes" and "ValuesPercentage" options.

hugepages plugin: Use the "complex" config callback.

hugepages plugin: Parse page size from the directory name.

hugepages plugin: Refactor the read_hugepage_entry() function.

* Move static variables into the "entry_info" struct.
* Turn flag into an actual flag, rather than a counter.
* Close "fh" as soon as possible.
* Return early if flags != HP_HAVE_ALL.
* Remove dead code (d_name *always* contains a dash).

hugepages plugin: Don't use pathconf(_PC_NAME_MAX).

Since we allocate the buffer on the stack, this doesn't make sense:
Best case, the returned value is the same as the PATH_MAX define. Worst
case, the returned value is larger and we create a stack overflow.

hugepages plugin: Run clang-format.

Merge remote-tracking branch 'github/pr/1931'

Tree wide: Unify initialization of multi-value value lists.

Use struct initializers to initialize the array like so:

  value_t values[] = {
    /* ... */
  };

Then initialize the value_list_t with:

  vl.values = values;
  vl.values_len = STATIC_ARRAY_SIZE (values);

contrib/examples/myplugin.c: Use a compound literal in the example plugin, too.

Tree wide: Use compound literals when dealing with value_t.

Merge branch 'collectd-5.6'

Conflicts:
src/collectd.conf.in

Merge branch 'collectd-5.5' into collectd-5.6

Merge remote-tracking branch 'github/pr/1930'

regex match: snprintf → ssnprintf

ssnprintf() ensures null termination, which is especially important when using "%s" in the format.

Address review comments:
- Fix match bug.
- Use sfree everywhere.
- Better variable names.

Merge remote-tracking branch 'github/pr/1929'

curl_json plugin: Skip unexpected non-map values.

Assume, for example, the config `Key "*/foo"`. This config expects JSON
in the form:

    {
      "bar": {
        "foo": 1337
      }
    }

If the available JSON is instead:

    {
      "error_code": 0,
      "bar": {
        "foo": 1337
      }
    }

the code will take a look at the zero associated with "error_code" and
determine that a map (with key "foo") is expected instead. Previously
the code would continue, eventually calling `cj_get_type()` which
expects that `key->type` is a valid pointer, resulting in a segmentation
fault.

This patch does three things to ensure that this segmentation fault does
not happen again:

1.  `cj_get_type()` checks its argument to make sure it is valid before
    dereferencing any pointers.
2.  In case a non-map is found when a map is expected, the code will
    return instead of limping on.
3.  After calling `cj_cb_inc_array_index()`, which may update the key,
    make sure that it actually did and that key is valid now.

Fixes: #1896

Document MetaData.

Allow Match:Regex to match metadata.

Merge remote-tracking branch 'github/pr/1922'

Truncate strjoin arguments if > DATA_MAX_NAME_LEN

If strjoin() runs out of buffer space when joining strings, use the
remainder of available space rather than skipping additional strings

Reworked return() to avoid extra strlen() call

Merge remote-tracking branch 'github/pr/1915'

src/utils_format_graphite.c: Re-insert blank line.

Address more review comments:
- Locking fixes,
- Leak fixes.

Improve if statement readability with braces

Update collectd.conf manpage and address style issues

dpdkstats: Added errors and filter_result types.

Map flow and mac related counters.
Minor types fixes.

Change-Id: Ib35ce61eb3dd1d602cf901e4bab677f9f65398a3
Signed-off-by: Taras Chornyi <tarasx.chornyi@intel.com>

Merge remote-tracking branch 'github/pr/1927'

collectd.conf(5): Document the semantics of the "TypesDB" option in more detail.

Fixed: #1241

contrib/migrate-4-5.px: Don't convert "ps_code" and "ps_data" to derive.

Fixes: #1687

Add missing option to write_graphite example

Signed-off-by: Florian Forster <octo@collectd.org>

varnish plugin: Fix syntax error.

turbostat plugin: Use parse_value_file().

thermal plugin: Use parse_value_file().

entropy plugin: Use parse_value_file().

cpufreq plugin: Use parse_value_file().

conntrack plugin: Use parse_value_file().

battery plugin: Replace getvalue() with parse_value_file().

src/daemon/common.[ch]: Implement parse_value_file().

Fixes: #1914

Merge remote-tracking branch 'github/pr/1920'

Merge remote-tracking branch 'github/pr/1924'

Merge remote-tracking branch 'github/pr/1923'

battery plugin: Reimplement sysfs_file_to_buffer() using read_file_contents().

Address review comments:
- Add meta_data_as_string,
- Use utility function.
- Leak fixes.
- Minor optimization.

Address review comments:
- Use a global to allow config-less operation.
- Use utility function.

Address review comments:
- Code simplifications,
- Leak fixes,
- ERROR -> WARNING.

Tree wide: Don't document the user_data_t holds user data.

Tree wide: Use user_data_t compound literal where appropriate.

apache plugin: End statements with semi-colons.

If only I knew why this wasn't considered a syntax error …

Bump version to 5.6.0; Update ChangeLog.

Document Plugin:WriteLog configuration.

Add optional configuration to write_log; allow writing JSON.

Document MetaData and DeleteMetaData.

Allow replacing within and deleting metadata keys.

Document interpolation and DeleteMetaData.

Implement interpolation.

Allow deleting metadata keys.

onewire plugin - improve error messages

Actually print error messages, return value from OW API is either 0 or -1 - not very informative.

Add RFC3339 Zulu time functions

Add option to write_graphite: DropDuplicateFields

This option drops duplicate type and type_instance fields or duplicate
type and plugin fields.

example_host.memory.memory.cached becomes example_host.memory.cached
example_host.cpu.0.cpu.idle becomes example_host.cpu.0.idle

battery plugin: Add StateFS backend.

Pull-Request: #1795

Squashed commit of the following:

commit 8f4841f381a59e1f1afd3101f93d84079866e8fa
Author: rinigus <rinigus.git@gmail.com>
Date:   Wed Sep 7 11:10:27 2016 +0300

    battery plugin, statefs backend: use power to report values

commit bab1a17ca6d6b2a21c64eb7c5a2b0cdb658f50bc
Author: Rinigus <rinigus.git@gmail.com>
Date:   Tue Sep 6 23:53:51 2016 +0300

    battery plugin, statefs backend: adjusting type names and small corrections

commit 6428720c330f7773f3e335c8426edb9d2f8f3139
Author: Rinigus <rinigus.git@gmail.com>
Date:   Sat Sep 3 12:15:19 2016 +0300

    battery statefs backend: cleanup

commit 6039a12774d3933a88bcc4612c92699d3a816f97
Author: Rinigus <rinigus.git@gmail.com>
Date:   Sat Sep 3 07:45:45 2016 +0300

    battery plugin: clang-format for statefs backend

commit 55ec4952dbc5643af335ca9d9802df438bf4a006
Author: Rinigus <rinigus.git@gmail.com>
Date:   Sat Sep 3 07:37:52 2016 +0300

    battery plugin: adjusting read_statefs call

commit 4a41e8f1b88f58ef09b0c8702e752d9d8f793e38
Author: Rinigus <rinigus.git@gmail.com>
Date:   Fri Sep 2 23:43:49 2016 +0300

    battery: statefs backend saves as battery plugin

commit bbc25d26663cb727b884e3931e6a909a71dac160
Author: Rinigus <rinigus.git@gmail.com>
Date:   Fri Sep 2 23:19:56 2016 +0300

    battery_statefs: define read function for statefs backend

commit 352cf7d9c2146e0960b8b688dfafcbdd54a789f1
Author: Rinigus <rinigus.git@gmail.com>
Date:   Thu Sep 1 23:36:35 2016 +0300

    battery_statefs: incorporating into battery plugin

commit 65e9d416c028d919efc2d901b782bb465c82f56b
Merge: c9eca12 bb98f71
Author: Rinigus <rinigus.git@gmail.com>
Date:   Thu Sep 1 23:08:36 2016 +0300

    battery_statefs: merge updates from upstream

commit c9eca127455571e8adf1f5c5f4290e56b793040b
Merge: a6b4c85 6cf7955
Author: Rinigus <rinigus.git@gmail.com>
Date:   Sun Jul 24 14:29:57 2016 +0300

    fix configure.ac

commit a6b4c851ae3e257faad417737065aacba1db9303
Author: Rinigus <rinigus.git@gmail.com>
Date:   Mon Jul 18 23:51:44 2016 +0300

    return statements

commit d31678bb5b3d9707063986ace17e5d7ba03976b5
Author: Rinigus <rinigus.git@gmail.com>
Date:   Mon Jul 18 23:46:19 2016 +0300

    changes suggested by @rubenk

commit 56785979aa49319a0a4622ff7ece0cc1ad1fd9a6
Author: Rinigus <rinigus.git@gmail.com>
Date:   Mon Jul 18 23:40:51 2016 +0300

    changes suggested by @rubenk

commit 6cbf0d03a9404f560d346ea770e06e38ed11206f
Author: Rinigus <rinigus.git@gmail.com>
Date:   Fri Jul 15 10:03:58 2016 +0300

    statefs_battery config

Merge branch 'collectd-5.6'

Merge branch 'collectd-5.5' into collectd-5.6