diff --git a/src/apcups.c b/src/apcups.c
index fb2583e63a50b332bf4ef1eb19892b1b6e87a926..b9a500d00f64769b08d66d77f8e467330daf84d8 100644 (file)
--- a/src/apcups.c
+++ b/src/apcups.c
* Anthony Gialluca <tonyabg at charter.net>
**/
* Anthony Gialluca <tonyabg at charter.net>
**/
+/*
+ * FIXME: Don't know why but without this here atof() was not returning
+ * correct values for me. This is behavior that I don't understand and
+ * should be examined in closer detail.
+ */
+#include <stdlib.h>
+
#include "collectd.h"
#include "collectd.h"
-#include "common.h" /* rrd_update_file */
-#include "plugin.h" /* plugin_register, plugin_submit */
-#include "configfile.h" /* cf_register */
-
-/* FIXME: Check defines before including anything! */
-#include <stdarg.h>
-#include <unistd.h>
-#include <string.h>
-#include <strings.h>
-#include <signal.h>
-#include <ctype.h>
-#include <syslog.h>
-#include <limits.h>
-#include <pwd.h>
-#include <time.h>
+#include "common.h" /* rrd_update_file */
+#include "plugin.h" /* plugin_register, plugin_submit */
+#include "configfile.h" /* cf_register */
+#include "utils_debug.h"
#include <errno.h>
#include <errno.h>
-#include <fcntl.h>
-#include <setjmp.h>
-#include <termios.h>
-#include <netdb.h>
-#include <sys/ioctl.h>
-#include <sys/ipc.h>
-#include <sys/sem.h>
-#include <sys/shm.h>
-#include <sys/socket.h>
-#include <sys/types.h>
-#include <sys/time.h>
-#include <time.h>
-#include <sys/wait.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
+
+#if HAVE_SYS_TYPES_H
+# include <sys/types.h>
+#endif
+#if HAVE_SYS_SOCKET_H
+# include <sys/socket.h>
+#endif
+#if HAVE_NETDB_H
+# include <netdb.h>
+#endif
+
+#if HAVE_NETINET_IN_H
+# include <netinet/in.h>
+#endif
+
+#ifndef APCMAIN
+# define APCMAIN 0
+#endif
#define NISPORT 3551
#define NISPORT 3551
-#define _(String) (String)
-#define N_(String) (String)
#define MAXSTRING 256
#define MAXSTRING 256
-<<<<<<< .mine
-=======
-#define Error_abort0(fmd) generic_error_out(__FILE__, __LINE__, fmd)
->>>>>>> .r733
#define MODULE_NAME "apcups"
#define MODULE_NAME "apcups"
-
-/* Prototypes */
-<<<<<<< .mine
-=======
-static void generic_error_out(const char *, int , const char *, ...);
->>>>>>> .r733
-
/* Default values for contacting daemon */
/* Default values for contacting daemon */
-static char *host = "localhost";
-static int port = NISPORT;
-
-static struct sockaddr_in tcp_serv_addr; /* socket information */
-static int net_errno = 0; /* error number -- not yet implemented */
-static char *net_errmsg = NULL; /* pointer to error message */
-static char net_errbuf[256]; /* error message buffer for messages */
-
-<<<<<<< .mine
-struct sockaddr_in tcp_serv_addr; /* socket information */
-int net_errno = 0; /* error number -- not yet implemented */
-char *net_errmsg = NULL; /* pointer to error message */
-char net_errbuf[256]; /* error message buffer for messages */
-
-void (*error_cleanup) (void) = NULL;
+static char *global_host = "localhost";
+static int global_port = NISPORT;
-=======
->>>>>>> .r733
/*
* 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
-static char *volt_file_template = "apcups_volt-%s.rrd";
-static char *volt_ds_def[] =
-{
- "DS:linev:GAUGE:"COLLECTD_HEARTBEAT":0:250",
- "DS:outputv:GAUGE:"COLLECTD_HEARTBEAT":0:250",
- NULL
-};
-static int volt_ds_num = 2;
-
-static char *bvolt_file_template = "apcups_bvolt-%s.rrd";
+#if !APCMAIN
+static char *bvolt_file_template = "apcups/voltage-%s.rrd";
static char *bvolt_ds_def[] =
{
static char *bvolt_ds_def[] =
{
- "DS:battv:GAUGE:"COLLECTD_HEARTBEAT":0:100",
+ "DS:voltage:GAUGE:"COLLECTD_HEARTBEAT":0:U",
};
static int bvolt_ds_num = 1;
};
static int bvolt_ds_num = 1;
-static char *load_file_template = "apcups_load-%s.rrd";
+static char *load_file_template = "apcups/charge_percent.rrd";
static char *load_ds_def[] =
{
static char *load_ds_def[] =
{
- "DS:loadpct:GAUGE:"COLLECTD_HEARTBEAT":0:120",
+ "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-%s.rrd";
+static char *charge_file_template = "apcups/charge.rrd";
static char *charge_ds_def[] =
{
static char *charge_ds_def[] =
{
- "DS:bcharge:GAUGE:"COLLECTD_HEARTBEAT":0:100",
+ "DS:charge:GAUGE:"COLLECTD_HEARTBEAT":0:U",
};
static int charge_ds_num = 1;
};
static int charge_ds_num = 1;
-static char *time_file_template = "apcups_time-%s.rrd";
+static char *time_file_template = "apcups/time.rrd";
static char *time_ds_def[] =
{
"DS:timeleft:GAUGE:"COLLECTD_HEARTBEAT":0:100",
};
static int time_ds_num = 1;
static char *time_ds_def[] =
{
"DS:timeleft:GAUGE:"COLLECTD_HEARTBEAT":0:100",
};
static int time_ds_num = 1;
-static char *temp_file_template = "apcups_temp-%s.rrd";
+static char *temp_file_template = "apcups/temperature.rrd";
static char *temp_ds_def[] =
{
static char *temp_ds_def[] =
{
- "DS:itemp:GAUGE:"COLLECTD_HEARTBEAT":0:100",
+ /* -273.15 is absolute zero */
+ "DS:temperature:GAUGE:"COLLECTD_HEARTBEAT":-274:U",
};
static int temp_ds_num = 1;
};
static int temp_ds_num = 1;
-static char *freq_file_template = "apcups_freq-%s.rrd";
+static char *freq_file_template = "apcups/frequency-%s.rrd";
static char *freq_ds_def[] =
{
static char *freq_ds_def[] =
{
- "DS:linefreq:GAUGE:"COLLECTD_HEARTBEAT":0:65",
+ "DS:frequency:GAUGE:"COLLECTD_HEARTBEAT":0:U",
};
static int freq_ds_num = 1;
};
static int freq_ds_num = 1;
};
static int config_keys_num = 2;
};
static int config_keys_num = 2;
-#endif /* ifndef APCMAIN */
-
-<<<<<<< .mine
-struct apc_detail_s {
- int connected;
- float linev;
- float loadpct;
- float bcharge;
- float timeleft;
- float outputv;
- float itemp;
- float battv;
- float linefreq;
-=======
+#endif /* if APCMAIN */
+
struct apc_detail_s
{
struct apc_detail_s
{
- float linev;
- float loadpct;
- float bcharge;
- float timeleft;
- float outputv;
- float itemp;
- float battv;
- float linefreq;
->>>>>>> .r733
+ double linev;
+ double loadpct;
+ double bcharge;
+ double timeleft;
+ double outputv;
+ double itemp;
+ double battv;
+ double linefreq;
};
#define BIG_BUF 4096
};
#define BIG_BUF 4096
-<<<<<<< .mine
-
-#define BIG_BUF 5000
-
-/* Implement snprintf */
-int asnprintf(char *str, size_t size, const char *fmt, ...)
-{
-#ifdef HAVE_VSNPRINTF
- va_list arg_ptr;
- int len;
-
- va_start(arg_ptr, fmt);
- len = vsnprintf(str, size, fmt, arg_ptr);
- va_end(arg_ptr);
-
- str[size - 1] = 0;
- return len;
-
-#else
-
- va_list arg_ptr;
- int len;
- char *buf;
-
- buf = (char *)malloc(BIG_BUF);
-
- va_start(arg_ptr, fmt);
- len = vsprintf(buf, fmt, arg_ptr);
- va_end(arg_ptr);
-
- if (len >= BIG_BUF){
- syslog(LOG_ERR, "apcups: asnprintf(): Buffer overflow");
- return(0);
- }
-
- memcpy(str, buf, size);
- str[size - 1] = 0;
-
- free(buf);
- return len;
-#endif
-}
-
-/* Implement vsnprintf() */
-int avsnprintf(char *str, size_t size, const char *format, va_list ap)
-{
-#ifdef HAVE_VSNPRINTF
- int len;
-
- len = vsnprintf(str, size, format, ap);
- str[size - 1] = 0;
-
- return len;
-
-#else
-
- int len;
- char *buf;
-
- buf = (char *)malloc(BIG_BUF);
-
- len = vsprintf(buf, format, ap);
- if (len >= BIG_BUF){
- syslog(LOG_ERR, "apcups: avsnprintf(): Buffer overflow");
- return(0);
- }
-
- memcpy(str, buf, size);
- str[size - 1] = 0;
-
- free(buf);
- return len;
-#endif
-}
-
-=======
->>>>>>> .r733
-/*
-<<<<<<< .mine
-=======
- * Subroutine normally called by macro error_abort() to print
- * FATAL ERROR message and supplied error message
- */
-static void generic_error_out(const char *file, int line, const char *fmt, ...)
-{
- char buf[256];
- va_list arg_ptr;
- int i;
-
- snprintf(buf, sizeof(buf), _("FATAL ERROR in %s at line %d\n"), file, line);
- i = strlen(buf);
- va_start(arg_ptr, fmt);
- vsnprintf((char *)&buf[i], sizeof(buf) - i, (char *)fmt, arg_ptr);
- va_end(arg_ptr);
- fprintf(stdout, "%s", buf);
-
- exit(1);
-}
-
/*
/*
->>>>>>> .r733
* Read nbytes from the network.
* It is possible that the total bytes require in several
* read requests
*/
* 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)
+static int read_nbytes (int *fd, char *ptr, int nbytes)
{
{
- int nleft, nread;
+ int nleft;
+ int nread;
nleft = nbytes;
nleft = nbytes;
+ nread = -1;
- while (nleft > 0) {
- do {
- nread = read(fd, ptr, nleft);
- } while (nread == -1 && (errno == EINTR || errno == EAGAIN));
+ assert (*fd >= 0);
- if (nread <= 0) {
- net_errno = errno;
- return (nread); /* error, or EOF */
+ while ((nleft > 0) && (nread != 0))
+ {
+ nread = read (*fd, ptr, nleft);
+
+ if (nread == -1 && (errno == EINTR || errno == EAGAIN))
+ continue;
+
+ if (nread == -1)
+ {
+ *fd = -1;
+ syslog (LOG_ERR, "apcups plugin: write failed: %s", strerror (errno));
+ return (-1);
}
nleft -= nread;
ptr += nread;
}
}
nleft -= nread;
ptr += nread;
}
- return (nbytes - nleft); /* return >= 0 */
+ return (nbytes - nleft);
}
/*
* Write nbytes to the network.
* It may require several writes.
*/
}
/*
* Write nbytes to the network.
* It may require several writes.
*/
-static int write_nbytes(int fd, char *ptr, int nbytes)
+static int write_nbytes (int *fd, void *buf, int buflen)
{
{
- int nleft, nwritten;
+ int nleft;
+ int nwritten;
+ char *ptr;
- nleft = nbytes;
- while (nleft > 0) {
- nwritten = write(fd, ptr, nleft);
+ assert (buflen > 0);
+ assert (*fd >= 0);
- if (nwritten <= 0) {
- net_errno = errno;
- return (nwritten); /* error */
+ ptr = (char *) buf;
+
+ nleft = buflen;
+ while (nleft > 0)
+ {
+ nwritten = write (*fd, ptr, nleft);
+
+ if ((nwritten == -1) && ((errno == EAGAIN) || (errno == EINTR)))
+ continue;
+
+ if (nwritten == -1)
+ {
+ syslog (LOG_ERR, "Writing to socket failed: %s", strerror (errno));
+ *fd = -1;
+ return (-1);
}
nleft -= nwritten;
ptr += nwritten;
}
}
nleft -= nwritten;
ptr += nwritten;
}
- return (nbytes - nleft);
+ /* If we get here, (nleft <= 0) is true */
+ return (buflen);
}
/* Close the network connection */
}
/* Close the network connection */
-static void net_close (int sockfd)
+static void net_close (int *fd)
{
short pktsiz = 0;
{
short pktsiz = 0;
+ assert (*fd >= 0);
+
/* send EOF sentinel */
/* send EOF sentinel */
- write_nbytes(sockfd, (char *) &pktsiz, sizeof(short));
- close (sockfd);
+ write_nbytes (fd, &pktsiz, sizeof (short));
+
+ close (*fd);
+ *fd = -1;
}
}
* Returns -1 on error
* Returns socket file descriptor otherwise
*/
* Returns -1 on error
* Returns socket file descriptor otherwise
*/
-static int net_open(char *host, char *service, int port)
+static int net_open (char *host, char *service, int port)
{
{
- int sockfd;
- struct hostent *hp;
- unsigned int inaddr; /* Careful here to use unsigned int for */
- /* compatibility with Alpha */
-
- /*
- * Fill in the structure serv_addr with the address of the server that
- * we want to connect with.
- */
- memset((char *)&tcp_serv_addr, 0, sizeof(tcp_serv_addr));
- tcp_serv_addr.sin_family = AF_INET;
- tcp_serv_addr.sin_port = htons(port);
+ int sd;
+ int status;
+ char port_str[8];
+ struct addrinfo ai_hints;
+ struct addrinfo *ai_return;
+ struct addrinfo *ai_list;
- if ((inaddr = inet_addr(host)) != INADDR_NONE) {
- tcp_serv_addr.sin_addr.s_addr = inaddr;
- } else {
- if ((hp = gethostbyname(host)) == NULL) {
- net_errmsg = "tcp_open: hostname error\n";
- return -1;
- }
+ assert ((port > 0x00000000) && (port <= 0x0000FFFF));
- if (hp->h_length != sizeof(inaddr) || hp->h_addrtype != AF_INET) {
- net_errmsg = "tcp_open: funny gethostbyname value\n";
- return -1;
- }
+ /* Convert the port to a string */
+ snprintf (port_str, 8, "%i", port);
+ port_str[7] = '\0';
+
+ /* Resolve name */
+ memset ((void *) &ai_hints, '\0', sizeof (ai_hints));
+ ai_hints.ai_family = AF_INET; /* XXX: Change this to `AF_UNSPEC' if apcupsd can handle IPv6 */
+ ai_hints.ai_socktype = SOCK_STREAM;
- tcp_serv_addr.sin_addr.s_addr = *(unsigned int *)hp->h_addr;
+ status = getaddrinfo (host, port_str, &ai_hints, &ai_return);
+ if (status != 0)
+ {
+ DBG ("getaddrinfo failed: %s", status == EAI_SYSTEM ? strerror (errno) : gai_strerror (status));
+ return (-1);
}
}
- /* Open a TCP socket */
- if ((sockfd = socket(AF_INET, SOCK_STREAM, 0)) < 0) {
- net_errmsg = "tcp_open: cannot open stream socket\n";
- return -1;
+ /* Create socket */
+ sd = -1;
+ for (ai_list = ai_return; ai_list != NULL; ai_list = ai_list->ai_next)
+ {
+ sd = socket (ai_list->ai_family, ai_list->ai_socktype, ai_list->ai_protocol);
+ if (sd >= 0)
+ break;
}
}
+ /* `ai_list' still holds the current description of the socket.. */
- /* connect to server */
-#if defined HAVE_OPENBSD_OS || defined HAVE_FREEBSD_OS
- /*
- * Work around a bug in OpenBSD & FreeBSD userspace pthreads
- * implementations. Rationale is the same as described above.
- */
- fcntl(sockfd, F_SETFL, fcntl(sockfd, F_GETFL));
-#endif
+ if (sd < 0)
+ {
+ DBG ("Unable to open a socket");
+ freeaddrinfo (ai_return);
+ return (-1);
+ }
+
+ status = connect (sd, ai_list->ai_addr, ai_list->ai_addrlen);
+
+ freeaddrinfo (ai_return);
- if (connect(sockfd, (struct sockaddr *)&tcp_serv_addr, sizeof(tcp_serv_addr)) < 0) {
- snprintf(net_errbuf, sizeof(net_errbuf),
- _("tcp_open: cannot connect to server %s on port %d.\n"
- "ERR=%s\n"), host, port, strerror(errno));
- net_errmsg = net_errbuf;
- close(sockfd);
- return -1;
+ if (status != 0) /* `connect(2)' failed */
+ {
+ DBG ("connect failed: %s", strerror (errno));
+ return (-1);
}
}
- return sockfd;
-} /* int net_open(char *host, char *service, int port) */
+ return (sd);
+} /* int net_open (char *host, char *service, int port) */
/*
* Receive a message from the other end. Each message consists of
/*
* Receive a message from the other end. Each message consists of
* 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 *buff, int maxlen)
+static int net_recv (int *sockfd, char *buf, int buflen)
{
{
- int nbytes;
+ int nbytes;
short pktsiz;
/* get data size -- in short */
short pktsiz;
/* get data size -- in short */
- if ((nbytes = read_nbytes(sockfd, (char *)&pktsiz, sizeof(short))) <= 0) {
- /* probably pipe broken because client died */
- return -1; /* assume hard EOF received */
- }
- if (nbytes != sizeof(short))
- return -2;
+ if ((nbytes = read_nbytes (sockfd, (char *) &pktsiz, sizeof (short))) <= 0)
+ return (-1);
+
+ if (nbytes != sizeof (short))
+ return (-2);
- pktsiz = ntohs(pktsiz); /* decode no. of bytes that follow */
- if (pktsiz > maxlen) {
- net_errmsg = "net_recv: record length too large\n";
- return -2;
+ pktsiz = ntohs (pktsiz);
+ if (pktsiz > buflen)
+ {
+ DBG ("record length too large");
+ return (-2);
}
}
+
if (pktsiz == 0)
if (pktsiz == 0)
- return 0; /* soft EOF */
+ return (0);
/* now read the actual data */
/* now read the actual data */
- if ((nbytes = read_nbytes(sockfd, buff, pktsiz)) <= 0) {
- net_errmsg = "net_recv: read_nbytes error\n";
- return -2;
- }
- if (nbytes != pktsiz) {
- net_errmsg = "net_recv: error in read_nbytes\n";
- return -2;
- }
+ if ((nbytes = read_nbytes (sockfd, buf, pktsiz)) <= 0)
+ return (-2);
- return (nbytes); /* return actual length of message */
-}
+ if (nbytes != pktsiz)
+ return (-2);
+
+ return (nbytes);
+} /* static int net_recv (int sockfd, char *buf, int buflen) */
/*
* Send a message over the network. The send consists of
* two network packets. The first is sends a short containing
* the length of the data packet which follows.
/*
* Send a message over the network. The send consists of
* two network packets. The first is sends a short containing
* the length of the data packet which follows.
- * Returns number of bytes sent
- * Returns -1 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;
{
int rc;
- short pktsiz;
+ short packet_size;
+
+ assert (len > 0);
/* send short containing size of data packet */
/* send short containing size of data packet */
- pktsiz = htons((short)len);
- rc = write_nbytes(sockfd, (char *)&pktsiz, sizeof(short));
- if (rc != sizeof(short)) {
- net_errmsg = "net_send: write_nbytes error of length prefix\n";
- return -1;
- }
+ packet_size = htons ((short) len);
+
+ rc = write_nbytes (sockfd, &packet_size, sizeof (packet_size));
+ if (rc != sizeof (packet_size))
+ return (-1);
/* send data packet */
/* send data packet */
- rc = write_nbytes(sockfd, buff, len);
- if (rc != len) {
- net_errmsg = "net_send: write_nbytes error\n";
- return -1;
- }
+ rc = write_nbytes (sockfd, buff, len);
+ if (rc != len)
+ return (-1);
- return rc;
+ return (0);
}
}
-
/* Get and print status from apcupsd NIS server */
/* Get and print status from apcupsd NIS server */
-static int do_pthreads_status(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)
{
{
-<<<<<<< .mine
- int sockfd, n;
- char recvline[MAXSTRING + 1];
- char *tokptr;
-
- if ((sockfd = net_open(host, NULL, port)) < 0){
- syslog(LOG_ERR, "apcups: Cannot open connection: %s",
- net_errmsg);
- net_errmsg = NULL;
- return;
- }
-
- net_send(sockfd, "status", 6);
-
- while ((n = net_recv(sockfd, recvline, sizeof(recvline))) > 0) {
- recvline[n] = 0;
-#ifdef APCMAIN
- fputs(recvline, stdout);
-#endif /* ifdef APCMAIN */
-=======
- int sockfd;
int n;
int n;
- char recvline[MAXSTRING + 1];
+ char recvline[1024];
char *tokptr;
char *key;
double value;
char *tokptr;
char *key;
double value;
+
+ static int sockfd = -1;
+ static unsigned int complain = 0;
+
#if APCMAIN
# define PRINT_VALUE(name, val) printf(" Found property: name = %s; value = %f;\n", name, val)
#else
# define PRINT_VALUE(name, val) /**/
#endif
#if APCMAIN
# define PRINT_VALUE(name, val) printf(" Found property: name = %s; value = %f;\n", name, val)
#else
# define PRINT_VALUE(name, val) /**/
#endif
->>>>>>> .r733
-
-<<<<<<< .mine
- tokptr = strtok(recvline,":");
- while(tokptr != NULL) {
- /* Look for Limit_Add */
- if(strncmp("LINEV",tokptr,5) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found LINEV.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->linev = atof (tokptr);
- }else if(strncmp("BATTV",tokptr,5) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found BATTV.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->battv = atof (tokptr);
- }else if(strncmp("ITEMP",tokptr,5) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found ITEMP.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->itemp = atof (tokptr);
- }else if(strncmp("LOADPCT",tokptr,7) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found LOADPCT.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->loadpct = atof (tokptr);
- }else if(strncmp("BCHARGE",tokptr,7) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found BCHARGE.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->bcharge = atof (tokptr);
- }else if(strncmp("OUTPUTV",tokptr,7) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found OUTPUTV.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->outputv = atof (tokptr);
- }else if(strncmp("LINEFREQ",tokptr,8) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found LINEFREQ.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->linefreq = atof (tokptr);
- }else if(strncmp("TIMELEFT",tokptr,8) == 0) {
-#ifdef APCMAIN
- fprintf(stdout," Found TIMELEFT.\n");
-#endif /* ifdef APCMAIN */
- tokptr = strtok(NULL," \t");
- if(tokptr == NULL) continue;
- apcups_detail->timeleft = atof (tokptr);
- } /* */
- tokptr = strtok(NULL,":");
- }
- }
-=======
- /* TODO: Keep the socket open, if possible */
- if ((sockfd = net_open (host, NULL, port)) < 0)
+
+ if (sockfd < 0)
+ {
+ if ((sockfd = net_open (host, NULL, port)) < 0)
+ {
+ /* Complain first time and once every six hours. */
+ int complain_step = 21600 / atoi (COLLECTD_STEP);
+
+ if (complain == 0 || (complain % complain_step) == 0)
+ syslog (LOG_ERR, "apcups plugin: Connecting to the apcupsd failed.");
+ complain++;
+
+ return (-1);
+ } else {
+ if(complain > 1)
+ syslog (LOG_ERR, "apcups plugin: Connection re-established to the apcupsd.");
+ }
+ complain = 0;
+ }
+
+ if (net_send (&sockfd, "status", 6) < 0)
{
{
- syslog (LOG_ERR, "apcups plugin: Connecting to the apcupsd failed.");
+ syslog (LOG_ERR, "apcups plugin: Writing to the socket failed.");
return (-1);
}
return (-1);
}
->>>>>>> .r733
-
-<<<<<<< .mine
- if (n < 0) {
- syslog(LOG_ERR, "apcups: Error recieving data: %s",
- net_errmsg);
- net_errmsg = NULL;
- return;
- }
- /* signal that we did in fact connect. */
- apcups_detail->connected = 1;
-
- net_close(sockfd);
-=======
- net_send (sockfd, "status", 6);
- while ((n = net_recv (sockfd, recvline, sizeof (recvline))) > 0)
+ /* XXX: Do we read `n' or `n-1' bytes? */
+ while ((n = net_recv (&sockfd, recvline, sizeof (recvline) - 1)) > 0)
{
{
+ assert (n < sizeof (recvline));
recvline[n] = '\0';
#if APCMAIN
printf ("net_recv = `%s';\n", recvline);
recvline[n] = '\0';
#if APCMAIN
printf ("net_recv = `%s';\n", recvline);
@@ -639,322 +432,192 @@ static int do_pthreads_status(char *host, int port, struct apc_detail_s *apcups_
apcups_detail->linefreq = value;
else if (strcmp ("TIMELEFT", tokptr) == 0)
apcups_detail->timeleft = value;
apcups_detail->linefreq = value;
else if (strcmp ("TIMELEFT", tokptr) == 0)
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) */
}
-
+
if (n < 0)
if (n < 0)
- Error_abort0(net_errmsg);
+ {
+ syslog (LOG_WARNING, "apcups plugin: Error reading from socket");
+ return (-1);
+ } else {
+ /* close the opened socket */
+ net_close(&sockfd);
+ }
- net_close(sockfd);
return (0);
return (0);
->>>>>>> .r733
}
}
-#ifdef APCMAIN
-int main(int argc, char **argv)
+#if APCMAIN
+/*
+ * This is used for testing apcups in a standalone mode.
+ * Usefull for debugging.
+ */
+int main (int argc, char **argv)
{
/* 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;
-<<<<<<< .mine
- 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";
-
- do_pthreads_status(host, port, &apcups_detail);
-
- return 0;
-=======
- 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";
- do_pthreads_status(host, port, &apcups_detail);
+ if(apc_query_server (global_host, global_port, &apcups_detail) < 0)
+ {
+ printf("apcups: Failed...\n");
+ return(-1);
+ }
return 0;
return 0;
->>>>>>> .r733
}
#else
}
#else
-static void apcups_init (void)
-{
- return;
-}
-
static int apcups_config (char *key, char *value)
{
static int apcups_config (char *key, char *value)
{
- static char lhost[126];
-
- if (strcasecmp (key, "host") == 0)
- {
- lhost[0] = '\0';
- strcpy(lhost,key);
- host = lhost;
- }
- else if (strcasecmp (key, "Port") == 0)
- {
- int port_tmp = atoi (value);
- if(port_tmp < 1 || port_tmp > 65535) {
- syslog (LOG_WARNING, "apcups: `port' failed: %s",
- value);
- return (1);
- } else {
- port = port_tmp;
- }
- }
- else
- {
- return (-1);
- }
- return(0);
+ if (strcasecmp (key, "host") == 0)
+ {
+ if (global_host != NULL)
+ {
+ free (global_host);
+ global_host = NULL;
+ }
+ if ((global_host = strdup (value)) == NULL)
+ return (1);
+ }
+ else if (strcasecmp (key, "Port") == 0)
+ {
+ int port_tmp = atoi (value);
+ if (port_tmp < 1 || port_tmp > 65535)
+ {
+ syslog (LOG_WARNING, "apcups plugin: Invalid port: %i", port_tmp);
+ return (1);
+ }
+ global_port = port_tmp;
+ }
+ else
+ {
+ return (-1);
+ }
+ return (0);
}
}
-#define BUFSIZE 256
-static void apcups_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apcups_init (void)
{
{
- char buf[BUFSIZE];
-
- if (snprintf (buf, BUFSIZE, "%u:%f:%f",
- (unsigned int) curtime,
- apcups_detail->linev,
- apcups_detail->outputv) >= BUFSIZE)
- return;
-
- plugin_submit (MODULE_NAME, host, buf);
+ return;
}
}
-static void apc_bvolt_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apc_write_voltage (char *host, char *inst, char *val)
{
{
- char buf[BUFSIZE];
+ char file[512];
+ int status;
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->battv) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_bvolt", host, buf);
-}
-
-static void apc_load_submit (char *host,
- struct apc_detail_s *apcups_detail)
-{
- char buf[BUFSIZE];
+ status = snprintf (file, 512, bvolt_file_template, inst);
+ if ((status < 1) || (status >= 512))
+ return;
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->loadpct) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_load", host, buf);
+ rrd_update_file (host, file, val, bvolt_ds_def, bvolt_ds_num);
}
}
-static void apc_charge_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apc_write_charge (char *host, char *inst, char *val)
{
{
- char buf[BUFSIZE];
-
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->bcharge) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_charge", host, buf);
+ rrd_update_file (host, charge_file_template, val, charge_ds_def, charge_ds_num);
}
}
-static void apc_temp_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apc_write_percent (char *host, char *inst, char *val)
{
{
- char buf[BUFSIZE];
-
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->itemp) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_temp", host, buf);
+ rrd_update_file (host, load_file_template, val, load_ds_def, load_ds_num);
}
}
-static void apc_time_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apc_write_timeleft (char *host, char *inst, char *val)
{
{
- char buf[BUFSIZE];
-
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->timeleft) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_time", host, buf);
+ rrd_update_file (host, time_file_template, val, time_ds_def, time_ds_num);
}
}
-static void apc_freq_submit (char *host,
- struct apc_detail_s *apcups_detail)
+static void apc_write_temperature (char *host, char *inst, char *val)
{
{
- char buf[BUFSIZE];
-
- if (snprintf (buf, BUFSIZE, "%u:%f",
- (unsigned int) curtime,
- apcups_detail->linefreq) >= BUFSIZE)
- return;
-
- plugin_submit ("apcups_freq", host, buf);
+ rrd_update_file (host, temp_file_template, val, temp_ds_def, temp_ds_num);
}
}
-#undef BUFSIZE
-static void apcups_read (void)
+static void apc_write_frequency (char *host, char *inst, char *val)
{
{
- struct apc_detail_s apcups_detail;
-
- apcups_detail.linev = 0.0;
- apcups_detail.loadpct = 0.0;
- apcups_detail.bcharge = 0.0;
- apcups_detail.timeleft = 0.0;
- apcups_detail.outputv = 0.0;
- apcups_detail.itemp = 0.0;
- apcups_detail.battv = 0.0;
- apcups_detail.linefreq = 0.0;
- apcups_detail.connected = 0;
-
-
- if (!*host || strcmp(host, "0.0.0.0") == 0)
- host = "localhost";
-
- do_pthreads_status(host, port, &apcups_detail);
-
- /*
- * if we did not connect then do not bother submitting
- * zeros. We want rrd files to have NAN.
- */
- if(!apcups_detail.connected) return;
-
- apcups_submit (host, &apcups_detail);
- apc_bvolt_submit (host, &apcups_detail);
- apc_load_submit (host, &apcups_detail);
- apc_charge_submit (host, &apcups_detail);
- apc_temp_submit (host, &apcups_detail);
- apc_time_submit (host, &apcups_detail);
- apc_freq_submit (host, &apcups_detail);
-}
+ char file[512];
+ int status;
+ status = snprintf (file, 512, freq_file_template, inst);
+ if ((status < 1) || (status >= 512))
+ return;
-static void apcups_write (char *host, char *inst, char *val)
-{
- char file[512];
- int status;
-
- status = snprintf (file, 512, volt_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, volt_ds_def, volt_ds_num);
+ rrd_update_file (host, file, val, freq_ds_def, freq_ds_num);
}
}
-static void apc_bvolt_write (char *host, char *inst, char *val)
+static void apc_submit_generic (char *type, char *inst,
+ double value)
{
{
- char file[512];
- int status;
-
- status = snprintf (file, 512, bvolt_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, bvolt_ds_def, bvolt_ds_num);
-}
+ char buf[512];
+ int status;
-static void apc_load_write (char *host, char *inst, char *val)
-{
- char file[512];
- int status;
-
- status = snprintf (file, 512, load_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, load_ds_def, load_ds_num);
-}
+ status = snprintf (buf, 512, "%u:%f",
+ (unsigned int) curtime, value);
+ if ((status < 1) || (status >= 512))
+ return;
-static void apc_charge_write (char *host, char *inst, char *val)
-{
- char file[512];
- int status;
-
- status = snprintf (file, 512, charge_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, charge_ds_def, charge_ds_num);
+ plugin_submit (type, inst, buf);
}
}
-static void apc_temp_write (char *host, char *inst, char *val)
+static void apc_submit (struct apc_detail_s *apcups_detail)
{
{
- char file[512];
- int status;
-
- status = snprintf (file, 512, temp_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, temp_ds_def, temp_ds_num);
+ apc_submit_generic ("apcups_voltage", "input", apcups_detail->linev);
+ apc_submit_generic ("apcups_voltage", "output", apcups_detail->outputv);
+ apc_submit_generic ("apcups_voltage", "battery", apcups_detail->battv);
+ apc_submit_generic ("apcups_charge", "-", apcups_detail->bcharge);
+ apc_submit_generic ("apcups_charge_pct", "-", apcups_detail->loadpct);
+ apc_submit_generic ("apcups_timeleft", "-", apcups_detail->timeleft);
+ apc_submit_generic ("apcups_temp", "-", apcups_detail->itemp);
+ apc_submit_generic ("apcups_frequency", "input", apcups_detail->linefreq);
}
}
-static void apc_time_write (char *host, char *inst, char *val)
+static void apcups_read (void)
{
{
- char file[512];
- int status;
-
- status = snprintf (file, 512, time_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
-
- rrd_update_file (host, file, val, time_ds_def, time_ds_num);
-}
+ struct apc_detail_s apcups_detail;
+ int status;
-static void apc_freq_write (char *host, char *inst, char *val)
-{
- char file[512];
- int status;
-
- status = snprintf (file, 512, freq_file_template, inst);
- if (status < 1)
- return;
- else if (status >= 512)
- return;
+ if (global_host == NULL)
+ return;
+
+ apcups_detail.linev = -1.0;
+ apcups_detail.outputv = -1.0;
+ apcups_detail.battv = -1.0;
+ apcups_detail.loadpct = -1.0;
+ apcups_detail.bcharge = -1.0;
+ apcups_detail.timeleft = -1.0;
+ apcups_detail.itemp = -300.0;
+ apcups_detail.linefreq = -1.0;
- rrd_update_file (host, file, val, freq_ds_def, freq_ds_num);
-}
+ status = apc_query_server (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)
+ return;
+
+ apc_submit (&apcups_detail);
+} /* apcups_read */
void module_register (void)
{
void module_register (void)
{
- plugin_register (MODULE_NAME, apcups_init, apcups_read, apcups_write);
- plugin_register ("apcups_bvolt", NULL, NULL, apc_bvolt_write);
- plugin_register ("apcups_load", NULL, NULL, apc_load_write);
- plugin_register ("apcups_charge", NULL, NULL, apc_charge_write);
- plugin_register ("apcups_temp", NULL, NULL, apc_temp_write);
- plugin_register ("apcups_time", NULL, NULL, apc_time_write);
- plugin_register ("apcups_freq", NULL, NULL, apc_freq_write);
+ plugin_register (MODULE_NAME, apcups_init, apcups_read, NULL);
+ plugin_register ("apcups_voltage", NULL, NULL, apc_write_voltage);
+ plugin_register ("apcups_charge", NULL, NULL, apc_write_charge);
+ plugin_register ("apcups_charge_pct", NULL, NULL, apc_write_percent);
+ plugin_register ("apcups_timeleft", NULL, NULL, apc_write_timeleft);
+ plugin_register ("apcups_temp", NULL, NULL, apc_write_temperature);
+ plugin_register ("apcups_frequency", NULL, NULL, apc_write_frequency);
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