diff --git a/src/apcups.c b/src/apcups.c
index 8cad3dff23e31726e07558507c182bc906913265..cd64426afa01e42125f3b210b7e39dc40f991c12 100644 (file)
--- a/src/apcups.c
+++ b/src/apcups.c
#if HAVE_NETDB_H
# include <netdb.h>
#endif
#if HAVE_NETDB_H
# include <netdb.h>
#endif
+
#if HAVE_NETINET_IN_H
# include <netinet/in.h>
#endif
#if HAVE_NETINET_IN_H
# include <netinet/in.h>
#endif
+#ifndef APCMAIN
+# define APCMAIN 0
+#endif
+
#define NISPORT 3551
#define MAXSTRING 256
#define MODULE_NAME "apcups"
#define NISPORT 3551
#define MAXSTRING 256
#define MODULE_NAME "apcups"
+#define APCUPS_DEFAULT_HOST "localhost"
+
/* Default values for contacting daemon */
static char *global_host = NULL;
static int global_port = NISPORT;
/* Default values for contacting daemon */
static char *global_host = NULL;
static int global_port = NISPORT;
/*
* The following are only if not compiled to test the module with its own main.
*/
/*
* The following are only if not compiled to test the module with its own main.
*/
-/* FIXME: Rename DSes to be more generic and follow established conventions. */
-#ifndef APCMAIN
+#if !APCMAIN
static char *bvolt_file_template = "apcups/voltage-%s.rrd";
static char *bvolt_ds_def[] =
{
static char *bvolt_file_template = "apcups/voltage-%s.rrd";
static char *bvolt_ds_def[] =
{
};
static int bvolt_ds_num = 1;
};
static int bvolt_ds_num = 1;
-static char *load_file_template = "apcups/charge_percent.rrd";
+static char *load_file_template = "apcups/load_percent.rrd";
static char *load_ds_def[] =
{
static char *load_ds_def[] =
{
- "DS:percent:GAUGE:"COLLECTD_HEARTBEAT":0:100",
+ "DS:percent:GAUGE:"COLLECTD_HEARTBEAT":0:110",
};
static int load_ds_num = 1;
};
static int load_ds_num = 1;
-static char *charge_file_template = "apcups/charge.rrd";
+static char *charge_file_template = "apcups/charge_percent.rrd";
static char *charge_ds_def[] =
{
static char *charge_ds_def[] =
{
- "DS:charge:GAUGE:"COLLECTD_HEARTBEAT":0:U",
+ "DS:percent:GAUGE:"COLLECTD_HEARTBEAT":0:110",
};
static int charge_ds_num = 1;
};
static int charge_ds_num = 1;
-static char *time_file_template = "apcups/time.rrd";
+static char *time_file_template = "apcups/timeleft.rrd";
static char *time_ds_def[] =
{
"DS:timeleft:GAUGE:"COLLECTD_HEARTBEAT":0:100",
static char *time_ds_def[] =
{
"DS:timeleft:GAUGE:"COLLECTD_HEARTBEAT":0:100",
static char *temp_ds_def[] =
{
/* -273.15 is absolute zero */
static char *temp_ds_def[] =
{
/* -273.15 is absolute zero */
- "DS:temperature:GAUGE:"COLLECTD_HEARTBEAT":-274:U",
+ "DS:value:GAUGE:"COLLECTD_HEARTBEAT":-274:U",
};
static int temp_ds_num = 1;
};
static int temp_ds_num = 1;
};
static int config_keys_num = 2;
};
static int config_keys_num = 2;
-#endif /* ifndef APCMAIN */
+#endif /* if APCMAIN */
struct apc_detail_s
{
struct apc_detail_s
{
#define BIG_BUF 4096
#define BIG_BUF 4096
-/*
- * Read nbytes from the network.
- * It is possible that the total bytes require in several
- * read requests
- */
-static int read_nbytes (int fd, char *ptr, int nbytes)
-{
- int nleft, nread;
-
- nleft = nbytes;
-
- while (nleft > 0) {
- do {
- nread = read (fd, ptr, nleft);
- } while (nread == -1 && (errno == EINTR || errno == EAGAIN));
-
- if (nread <= 0) {
- return (nread); /* error, or EOF */
- }
-
- nleft -= nread;
- ptr += nread;
- }
-
- return (nbytes - nleft); /* return >= 0 */
-}
-
-/*
- * Write nbytes to the network.
- * It may require several writes.
- */
-static int write_nbytes (int fd, void *buf, int buflen)
+#if APCMAIN
+/* Close the network connection */
+static void net_close (int *fd)
{
{
- int nleft;
- int nwritten;
- char *ptr;
-
- ptr = (char *) buf;
+ uint16_t packet_size = 0;
- nleft = buflen;
- while (nleft > 0)
- {
- nwritten = write (fd, ptr, nleft);
+ assert (*fd >= 0);
- if (nwritten <= 0)
- {
- syslog (LOG_ERR, "Writing to socket failed: %s", strerror (errno));
- return (nwritten);
- }
-
- nleft -= nwritten;
- ptr += nwritten;
- }
-
- /* If we get here, (nleft <= 0) is true */
- return (buflen);
-}
-
-/* Close the network connection */
-static void net_close (int sockfd)
-{
- short pktsiz = 0;
+ DBG ("Gracefully shutting down socket %i.", *fd);
/* send EOF sentinel */
/* send EOF sentinel */
- write_nbytes (sockfd, &pktsiz, sizeof (short));
- close (sockfd);
-}
+ swrite (*fd, (void *) &packet_size, sizeof (packet_size));
+ close (*fd);
+ *fd = -1;
+}
+#endif /* APCMAIN */
/*
* Open a TCP connection to the UPS network server
/*
* Open a TCP connection to the UPS network server
if (status != 0) /* `connect(2)' failed */
{
DBG ("connect failed: %s", strerror (errno));
if (status != 0) /* `connect(2)' failed */
{
DBG ("connect failed: %s", strerror (errno));
+ close (sd);
return (-1);
}
return (-1);
}
+ DBG ("Done opening a socket %i", sd);
+
return (sd);
} /* int net_open (char *host, char *service, int port) */
return (sd);
} /* int net_open (char *host, char *service, int port) */
* Returns -1 on hard end of file (i.e. network connection close)
* Returns -2 on error
*/
* Returns -1 on hard end of file (i.e. network connection close)
* Returns -2 on error
*/
-static int net_recv (int sockfd, char *buf, int buflen)
+static int net_recv (int *sockfd, char *buf, int buflen)
{
{
- int nbytes;
- short pktsiz;
+ uint16_t packet_size;
/* get data size -- in short */
/* get data size -- in short */
- if ((nbytes = read_nbytes (sockfd, (char *) &pktsiz, sizeof (short))) <= 0)
+ if (sread (*sockfd, (void *) &packet_size, sizeof (packet_size)) != 0)
+ {
+ *sockfd = -1;
return (-1);
return (-1);
+ }
- if (nbytes != sizeof (short))
- return (-2);
-
- pktsiz = ntohs (pktsiz);
- if (pktsiz > buflen)
+ packet_size = ntohs (packet_size);
+ if (packet_size > buflen)
{
DBG ("record length too large");
return (-2);
}
{
DBG ("record length too large");
return (-2);
}
- if (pktsiz == 0)
+ if (packet_size == 0)
return (0);
/* now read the actual data */
return (0);
/* now read the actual data */
- if ((nbytes = read_nbytes (sockfd, buf, pktsiz)) <= 0)
- return (-2);
-
- if (nbytes != pktsiz)
- return (-2);
+ if (sread (*sockfd, (void *) buf, packet_size) != 0)
+ {
+ *sockfd = -1;
+ return (-1);
+ }
- return (nbytes);
-} /* static int net_recv (int sockfd, char *buf, int buflen) */
+ return ((int) packet_size);
+} /* static int net_recv (int *sockfd, char *buf, int buflen) */
/*
* Send a message over the network. The send consists of
/*
* Send a message over the network. The send consists of
* Returns zero on success
* Returns non-zero on error
*/
* Returns zero on success
* Returns non-zero on error
*/
-static int net_send (int sockfd, char *buff, int len)
+static int net_send (int *sockfd, char *buff, int len)
{
{
- int rc;
- short packet_size;
+ uint16_t packet_size;
+
+ assert (len > 0);
+ assert (*sockfd >= 0);
/* send short containing size of data packet */
/* send short containing size of data packet */
- packet_size = htons ((short) len);
+ packet_size = htons ((uint16_t) len);
- rc = write_nbytes (sockfd, &packet_size, sizeof (packet_size));
- if (rc != sizeof (packet_size))
+ if (swrite (*sockfd, (void *) &packet_size, sizeof (packet_size)) != 0)
+ {
+ *sockfd = -1;
return (-1);
return (-1);
+ }
/* send data packet */
/* send data packet */
- rc = write_nbytes (sockfd, buff, len);
- if (rc != len)
- return (-1);
+ if (swrite (*sockfd, (void *) buff, len) != 0)
+ {
+ *sockfd = -1;
+ return (-2);
+ }
return (0);
}
return (0);
}
static int apc_query_server (char *host, int port,
struct apc_detail_s *apcups_detail)
{
static int apc_query_server (char *host, int port,
struct apc_detail_s *apcups_detail)
{
- int sockfd;
int n;
char recvline[1024];
char *tokptr;
char *key;
double value;
int n;
char recvline[1024];
char *tokptr;
char *key;
double value;
- static int complain = 0;
+ static int sockfd = -1;
+ static complain_t compl;
#if APCMAIN
# define PRINT_VALUE(name, val) printf(" Found property: name = %s; value = %f;\n", name, val)
#if APCMAIN
# define PRINT_VALUE(name, val) printf(" Found property: name = %s; value = %f;\n", name, val)
# define PRINT_VALUE(name, val) /**/
#endif
# define PRINT_VALUE(name, val) /**/
#endif
- /* TODO: Keep the socket open, if possible */
- if ((sockfd = net_open (host, NULL, port)) < 0)
+ if (sockfd < 0)
{
{
- /* Complain once every six hours. */
- int complain_step = 21600 / atoi (COLLECTD_STEP);
-
- if ((complain % complain_step) == 0)
- syslog (LOG_ERR, "apcups plugin: Connecting to the apcupsd failed.");
- complain++;
-
- return (-1);
+ if ((sockfd = net_open (host, NULL, port)) < 0)
+ {
+ plugin_complain (LOG_ERR, &compl, "apcups plugin: "
+ "Connecting to the apcupsd failed.");
+ return (-1);
+ }
+ else
+ {
+ plugin_relief (LOG_NOTICE, &compl, "apcups plugin: "
+ "Connection re-established to the apcupsd.");
+ }
}
}
- complain = 0;
- if (net_send (sockfd, "status", 6) < 0)
+ if (net_send (&sockfd, "status", 6) < 0)
{
syslog (LOG_ERR, "apcups plugin: Writing to the socket failed.");
return (-1);
}
{
syslog (LOG_ERR, "apcups plugin: Writing to the socket failed.");
return (-1);
}
- /* XXX: Do we read `n' or `n-1' bytes? */
- while ((n = net_recv (sockfd, recvline, sizeof (recvline) - 1)) > 0)
+ while ((n = net_recv (&sockfd, recvline, sizeof (recvline) - 1)) > 0)
{
assert (n < sizeof (recvline));
recvline[n] = '\0';
{
assert (n < sizeof (recvline));
recvline[n] = '\0';
printf ("net_recv = `%s';\n", recvline);
#endif /* if APCMAIN */
printf ("net_recv = `%s';\n", recvline);
#endif /* if APCMAIN */
- tokptr = strtok (recvline, ":");
+ tokptr = strtok (recvline, " :\t");
while (tokptr != NULL)
{
key = tokptr;
while (tokptr != NULL)
{
key = tokptr;
- if ((tokptr = strtok (NULL, " \t")) == NULL)
+ if ((tokptr = strtok (NULL, " :\t")) == NULL)
continue;
value = atof (tokptr);
continue;
value = atof (tokptr);
+
PRINT_VALUE (key, value);
if (strcmp ("LINEV", key) == 0)
apcups_detail->linev = value;
PRINT_VALUE (key, value);
if (strcmp ("LINEV", key) == 0)
apcups_detail->linev = value;
- else if (strcmp ("BATTV", tokptr) == 0)
+ else if (strcmp ("BATTV", key) == 0)
apcups_detail->battv = value;
apcups_detail->battv = value;
- else if (strcmp ("ITEMP", tokptr) == 0)
+ else if (strcmp ("ITEMP", key) == 0)
apcups_detail->itemp = value;
apcups_detail->itemp = value;
- else if (strcmp ("LOADPCT", tokptr) == 0)
+ else if (strcmp ("LOADPCT", key) == 0)
apcups_detail->loadpct = value;
apcups_detail->loadpct = value;
- else if (strcmp ("BCHARGE", tokptr) == 0)
+ else if (strcmp ("BCHARGE", key) == 0)
apcups_detail->bcharge = value;
apcups_detail->bcharge = value;
- else if (strcmp ("OUTPUTV", tokptr) == 0)
+ else if (strcmp ("OUTPUTV", key) == 0)
apcups_detail->outputv = value;
apcups_detail->outputv = value;
- else if (strcmp ("LINEFREQ", tokptr) == 0)
+ else if (strcmp ("LINEFREQ", key) == 0)
apcups_detail->linefreq = value;
apcups_detail->linefreq = value;
- else if (strcmp ("TIMELEFT", tokptr) == 0)
+ else if (strcmp ("TIMELEFT", key) == 0)
apcups_detail->timeleft = value;
apcups_detail->timeleft = value;
- else
- {
- syslog (LOG_WARNING, "apcups plugin: Received unknown property from apcupsd: `%s' = %f",
- key, value);
- }
tokptr = strtok (NULL, ":");
} /* while (tokptr != NULL) */
}
tokptr = strtok (NULL, ":");
} /* while (tokptr != NULL) */
}
-
- net_close (sockfd);
-
+
if (n < 0)
{
syslog (LOG_WARNING, "apcups plugin: Error reading from socket");
return (-1);
}
if (n < 0)
{
syslog (LOG_WARNING, "apcups plugin: Error reading from socket");
return (-1);
}
+#if APCMAIN
+ else
+ {
+ /* close the opened socket */
+ net_close (&sockfd);
+ }
+#endif /* APCMAIN */
return (0);
}
return (0);
}
-#ifdef APCMAIN
+#if APCMAIN
/*
* This is used for testing apcups in a standalone mode.
* Usefull for debugging.
/*
* This is used for testing apcups in a standalone mode.
* Usefull for debugging.
/* we are not really going to use this */
struct apc_detail_s apcups_detail;
/* we are not really going to use this */
struct apc_detail_s apcups_detail;
- openlog ("apcups", LOG_PID | LOG_NDELAY | LOG_LOCAL1);
+ openlog ("apcups", LOG_PID | LOG_NDELAY | LOG_LOCAL1, LOG_USER);
- if (!*host || strcmp (host, "0.0.0.0") == 0)
- host = "localhost";
+ if (global_host == NULL || strcmp (global_host, "0.0.0.0") == 0)
+ global_host = "localhost";
- if(do_apc_status (host, port, &apcups_detail) < 0)
+ if(apc_query_server (global_host, global_port, &apcups_detail) < 0)
{
printf("apcups: Failed...\n");
return(-1);
}
{
printf("apcups: Failed...\n");
return(-1);
}
- apc_query_server (global_host, global_port, &apcups_detail);
-
return 0;
}
#else
return 0;
}
#else
if ((status < 1) || (status >= 512))
return;
if ((status < 1) || (status >= 512))
return;
+ DBG ("plugin_submit (%s, %s, %s);", type, inst, buf);
plugin_submit (type, inst, buf);
}
plugin_submit (type, inst, buf);
}
struct apc_detail_s apcups_detail;
int status;
struct apc_detail_s apcups_detail;
int status;
- if (global_host == NULL)
- return;
-
apcups_detail.linev = -1.0;
apcups_detail.outputv = -1.0;
apcups_detail.battv = -1.0;
apcups_detail.linev = -1.0;
apcups_detail.outputv = -1.0;
apcups_detail.battv = -1.0;
apcups_detail.itemp = -300.0;
apcups_detail.linefreq = -1.0;
apcups_detail.itemp = -300.0;
apcups_detail.linefreq = -1.0;
- status = apc_query_server (global_host, global_port, &apcups_detail);
+ status = apc_query_server (global_host == NULL
+ ? APCUPS_DEFAULT_HOST
+ : global_host,
+ global_port, &apcups_detail);
/*
* if we did not connect then do not bother submitting
* zeros. We want rrd files to have NAN.
*/
if (status != 0)
/*
* if we did not connect then do not bother submitting
* zeros. We want rrd files to have NAN.
*/
if (status != 0)
+ {
+ DBG ("apc_query_server (%s, %i) = %i",
+ global_host == NULL
+ ? APCUPS_DEFAULT_HOST
+ : global_host,
+ global_port, status);
return;
return;
+ }
apc_submit (&apcups_detail);
} /* apcups_read */
apc_submit (&apcups_detail);
} /* apcups_read */
cf_register (MODULE_NAME, apcups_config, config_keys, config_keys_num);
}
cf_register (MODULE_NAME, apcups_config, config_keys, config_keys_num);
}
-#endif /* ifdef APCMAIN */
+#endif /* if APCMAIN */
#undef MODULE_NAME
#undef MODULE_NAME