index 1ea0f0d2000816521eaab1ca43131f45af52e564..3a41fdf65e0e7d628d9a7e032412fb8f6f9e8649 100644 (file)
* private helper functions
*/
+/* store the specified host-name (once per iteration) */
static int
-sdb_collectd_add_host(const char *hostname, sdb_time_t last_update)
+sdb_collectd_store_host(sdb_collectd_state_t *state,
+ const char *hostname, sdb_time_t last_update)
{
int status;
+ if (last_update > state->current_timestamp)
+ state->current_timestamp = last_update;
+
+ if (state->current_host && (! strcasecmp(state->current_host, hostname)))
+ return 0;
+ /* else: first/new host */
+
+ if (state->current_host) {
+ sdb_log(SDB_LOG_DEBUG, "collectd::unixsock backend: Added/updated "
+ "%i service%s (%i failed) for host '%s'.",
+ state->svc_updated, state->svc_updated == 1 ? "" : "s",
+ state->svc_failed, state->current_host);
+ state->svc_updated = state->svc_failed = 0;
+ free(state->current_host);
+ }
+
+ state->current_host = strdup(hostname);
+ if (! state->current_host) {
+ char errbuf[1024];
+ sdb_log(SDB_LOG_ERR, "collectd::unixsock backend: Failed to allocate "
+ "string buffer: %s",
+ sdb_strerror(errno, errbuf, sizeof(errbuf)));
+ return -1;
+ }
+
status = sdb_store_host(hostname, last_update);
if (status < 0) {
"host '%s' (last update timestamp = %"PRIscTIME").",
hostname, last_update);
return 0;
-} /* sdb_collectd_add_host */
+} /* sdb_collectd_store_host */
static int
sdb_collectd_add_svc(const char *hostname, const char *plugin,
type = data[3].data.string;
state = SDB_OBJ_WRAPPER(user_data)->data;
-
- if (! state->current_host) {
- state->current_host = strdup(hostname);
- state->current_timestamp = last_update;
- }
-
- if (! state->current_host) {
- char errbuf[1024];
- sdb_log(SDB_LOG_ERR, "collectd::unixsock backend: Failed to allocate "
- "string buffer: %s",
- sdb_strerror(errno, errbuf, sizeof(errbuf)));
+ if (sdb_collectd_store_host(state, hostname, last_update))
return -1;
- }
-
- if (! sdb_store_has_host(hostname))
- sdb_collectd_add_host(hostname, last_update);
if (sdb_collectd_add_svc(hostname, plugin, type, last_update))
++state->svc_failed;
else
++state->svc_updated;
-
- if (! strcasecmp(state->current_host, hostname)) {
- if (last_update > state->current_timestamp)
- state->current_timestamp = last_update;
- return 0;
- }
-
- /* new host */
- sdb_collectd_add_host(hostname, last_update);
-
- sdb_log(SDB_LOG_DEBUG, "collectd::unixsock backend: Added/updated "
- "%i service%s (%i failed) for host '%s'.",
- state->svc_updated, state->svc_updated == 1 ? "" : "s",
- state->svc_failed, state->current_host);
- state->svc_updated = state->svc_failed = 0;
-
- free(state->current_host);
- state->current_host = strdup(hostname);
- state->current_timestamp = last_update;
return 0;
} /* sdb_collectd_get_data */
static int
sdb_collectd_shutdown(__attribute__((unused)) sdb_object_t *user_data)
{
+ if (! user_data)
+ return -1;
+
+ sdb_unixsock_client_destroy(SDB_OBJ_WRAPPER(user_data)->data);
+ SDB_OBJ_WRAPPER(user_data)->data = NULL;
return 0;
} /* sdb_collectd_shutdown */
}
if (state.current_host) {
- sdb_collectd_add_host(state.current_host, state.current_timestamp);
sdb_log(SDB_LOG_DEBUG, "collectd::unixsock backend: Added/updated "
"%i service%s (%i failed) for host '%s'.",
state.svc_updated, state.svc_updated == 1 ? "" : "s",
state.svc_failed, state.current_host);
+ free(state.current_host);
}
return 0;
} /* sdb_collectd_collect */
{
int i;
+ if (! ci) /* nothing to do to deconfigure this plugin */
+ return 0;
+
for (i = 0; i < ci->children_num; ++i) {
oconfig_item_t *child = ci->children + i;
sdb_plugin_set_info(info, SDB_PLUGIN_INFO_VERSION, SDB_VERSION);
sdb_plugin_set_info(info, SDB_PLUGIN_INFO_PLUGIN_VERSION, SDB_VERSION);
- sdb_plugin_register_config("collectd::unixsock", sdb_collectd_config);
+ sdb_plugin_register_config(sdb_collectd_config);
return 0;
} /* sdb_version_extra */