Add thresholds support for check_cluster + lots of standardization.

Add forgotten items to NEWS.

git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1688 f882894a-f735-0410-b71e-b25c423dba1c

diff --git a/NEWS b/NEWS
index ed94cd3a9e82f6b66f2f2a0c387e88fd15754daa..6fb2468e1afa9eef1ade1be7e6302587797ea111 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,10 @@
 This file documents the major additions and syntax changes between releases.
 
+1.4.9 ??
+       Inclusion of contrib/check_cluster2 as check_cluster with some improvements
+       Fix the -S option in check_by_ssh
+       check_snmp now support Counter64
+
 1.4.8 11th April 2007
        Respects --without-world-permissions for setuid plugins
        check_disk extra options for regex matching of filesystems and grouping of filesystems

diff --git a/plugins/Makefile.am b/plugins/Makefile.am
index 3909dc5cfa25f4207a20b000d4fae36823d632e7..5e3cb0f17a2def83bf6d8369928f110bf54e9393 100644 (file)
--- a/plugins/Makefile.am
+++ b/plugins/Makefile.am
@@ -48,6 +48,7 @@ test:
 # the actual targets
 
 check_apt_LDADD = $(BASEOBJS) runcmd.o
+check_cluster_LDADD = $(BASEOBJS)
 check_dig_LDADD = $(NETLIBS) runcmd.o 
 check_disk_LDADD = $(BASEOBJS) popen.o
 check_dns_LDADD = $(NETLIBS) runcmd.o
@@ -90,7 +91,7 @@ negate_LDADD = $(BASEOBJS) popen.o
 urlize_LDADD = $(BASEOBJS) popen.o
 
 check_apt_DEPENDENCIES = check_apt.c $(BASEOBJS) runcmd.o $(DEPLIBS)
-check_cluster_DEPENDENCIES = check_cluster.c $(DEPLIBS)
+check_cluster_DEPENDENCIES = check_cluster.c $(BASEOBJS) $(DEPLIBS)
 check_dig_DEPENDENCIES = check_dig.c $(NETOBJS) runcmd.o $(DEPLIBS)
 check_disk_DEPENDENCIES = check_disk.c $(BASEOBJS) popen.o $(DEPLIBS) 
 check_dns_DEPENDENCIES = check_dns.c $(NETOBJS) runcmd.o $(DEPLIBS)

diff --git a/plugins/check_cluster.c b/plugins/check_cluster.c
index e60948d6d17ceb89a30c69c064e58b69c8b74704..98003f1f64ef0b54fba988e7a1fb26a33939846e 100644 (file)
--- a/plugins/check_cluster.c
+++ b/plugins/check_cluster.c
@@ -3,10 +3,11 @@
  * CHECK_CLUSTER2.C - Host and Service Cluster Plugin for Nagios 2.x
  *
  * Copyright (c) 2000-2004 Ethan Galstad (nagios@nagios.org)
+ * Copyright (c) 2007 nagios-plugins team
  * License: GPL
- * Last Modified:   03-11-2004
+ * Last Modified: $Date$
  *
- * License:
+ * License Information:
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -22,31 +23,21 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  *
- *****************************************************************************/
+ * $Id$
+ * 
+******************************************************************************/
 
+const char *progname = "check_cluster";
+const char *revision = "$Revision$";
+const char *copyright = "2000-2007";
+const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
-#include <getopt.h>
-
-#define OK             0
-#define ERROR          -1
-
-#define TRUE           1
-#define FALSE          0
+#include "common.h"
+#include "utils.h"
 
 #define CHECK_SERVICES 1
 #define CHECK_HOSTS    2
 
-#define MAX_INPUT_BUFFER       1024
-
-#define STATE_OK       0
-#define STATE_WARNING  1
-#define STATE_CRITICAL 2
-#define STATE_UNKNOWN  3
-
 int total_services_ok=0;
 int total_services_warning=0;
 int total_services_unknown=0;
@@ -56,14 +47,15 @@ int total_hosts_up=0;
 int total_hosts_down=0;
 int total_hosts_unreachable=0;
 
-int warning_threshold=1;
-int critical_threshold=1;
+char *warn_threshold;
+char *crit_threshold;
 
 int check_type=CHECK_SERVICES;
 
 char *data_vals=NULL;
 char *label=NULL;
 
+int verbose=0;
 
 int process_arguments(int,char **);
 
@@ -75,33 +67,15 @@ int main(int argc, char **argv){
        int data_val;
        int return_code=STATE_OK;
        int error=FALSE;
+       thresholds *thresholds;
 
-       if(process_arguments(argc,argv)==ERROR){
-
-               printf("Invalid arguments supplied\n");
-               printf("\n");
-
-               printf("Host/Service Cluster Plugin for Nagios 2\n");
-               printf("Copyright (c) 2000-2004 Ethan Galstad (nagios@nagios.org)\n");
-               printf("Last Modified: 03-11-2004\n");
-               printf("License: GPL\n");
-               printf("\n");
-               printf("Usage: %s (-s | -h) [-l label] [-w threshold] [-c threshold] [-d val1,val2,...,valn]\n",argv[0]);
-               printf("\n");
-               printf("Options:\n");
-               printf("   -s, --service  = Check service cluster status\n");
-               printf("   -h, --host     = Check host cluster status\n");
-               printf("   -l, --label    = Optional prepended text output (i.e. \"Host cluster\")\n");
-               printf("   -w, --warning  = Specifies the number of hosts or services in cluster that must be in\n");
-               printf("                    a non-OK state in order to return a WARNING status level\n");
-               printf("   -c, --critical = Specifies the number of hosts or services in cluster that must be in\n");
-               printf("                    a non-OK state in order to return a CRITICAL status level\n");
-               printf("   -d, --data     = The status codes of the hosts or services in the cluster, separated\n");
-               printf("                    by commas\n");
-               printf("\n");
-
-               return STATE_UNKNOWN;
-               }
+       if(process_arguments(argc,argv)==ERROR)
+               usage(_("Could not parse arguments"));
+
+       /* Initialize the thresholds */
+       set_thresholds(&thresholds, warn_threshold, crit_threshold);
+       if(verbose)
+               print_thresholds("check_cluster", thresholds);
 
        /* check the data values */
        for(ptr=strtok(data_vals,",");ptr!=NULL;ptr=strtok(NULL,",")){
@@ -146,22 +120,17 @@ int main(int argc, char **argv){
 
        /* return the status of the cluster */
        if(check_type==CHECK_SERVICES){
-               if((total_services_warning+total_services_unknown+total_services_critical) >= critical_threshold)
-                       return_code=STATE_CRITICAL;
-               else if((total_services_warning+total_services_unknown+total_services_critical) >= warning_threshold)
-                       return_code=STATE_WARNING;
-               else
-                       return_code=STATE_OK;
-               printf("%s %s: %d ok, %d warning, %d unknown, %d critical\n",(label==NULL)?"Service cluster":label,(return_code==STATE_OK)?"ok":"problem",total_services_ok,total_services_warning,total_services_unknown,total_services_critical);
+               return_code=get_status(total_services_warning+total_services_unknown+total_services_critical, thresholds);
+               printf("CLUSTER %s: %s: %d ok, %d warning, %d unknown, %d critical\n",
+                       state_text(return_code), (label==NULL)?"Service cluster":label,
+                       total_services_ok,total_services_warning,
+                       total_services_unknown,total_services_critical);
                 }
        else{
-               if((total_hosts_down+total_hosts_unreachable) >= critical_threshold)
-                       return_code=STATE_CRITICAL;
-               else if((total_hosts_down+total_hosts_unreachable) >= warning_threshold)
-                       return_code=STATE_WARNING;
-               else
-                       return_code=STATE_OK;
-               printf("%s %s: %d up, %d down, %d unreachable\n",(label==NULL)?"Host cluster":label,(return_code==STATE_OK)?"ok":"problem",total_hosts_up,total_hosts_down,total_hosts_unreachable);
+               return_code=get_status(total_hosts_down+total_hosts_unreachable, thresholds);
+               printf("CLUSTER %s: %s: %d up, %d down, %d unreachable\n",
+                       state_text(return_code), (label==NULL)?"Host cluster":label,
+                       total_hosts_up,total_hosts_down,total_hosts_unreachable);
                 }
 
        return return_code;
@@ -179,6 +148,8 @@ int process_arguments(int argc, char **argv){
                {"label",    required_argument,0,'l'},
                {"host",     no_argument,      0,'h'},
                {"service",  no_argument,      0,'s'},
+               {"verbose",  no_argument,      0,'v'},
+               {"help",     no_argument,      0,'H'},
                {0,0,0,0}
                };
 
@@ -188,7 +159,7 @@ int process_arguments(int argc, char **argv){
 
        while(1){
 
-               c=getopt_long(argc,argv,"hsw:c:d:l:",longopts,&option);
+               c=getopt_long(argc,argv,"hHsvw:c:d:l:",longopts,&option);
 
                if(c==-1 || c==EOF || c==1)
                        break;
@@ -204,11 +175,15 @@ int process_arguments(int argc, char **argv){
                        break;
 
                case 'w': /* warning threshold */
-                       warning_threshold=atoi(optarg);
+                       if (strspn (optarg, "0123456789:,") < strlen (optarg))
+                               usage2 (_("Invalid warning threshold: %s\n"), optarg);
+                       warn_threshold = strdup(optarg);
                        break;
 
                case 'c': /* warning threshold */
-                       critical_threshold=atoi(optarg);
+                       if (strspn (optarg, "0123456789:,") < strlen (optarg))
+                               usage2 (_("Invalid critical threshold: %s\n"), optarg);
+                       crit_threshold = strdup(optarg);
                        break;
 
                case 'd': /* data values */
@@ -219,6 +194,15 @@ int process_arguments(int argc, char **argv){
                        label=(char *)strdup(optarg);
                        break;
 
+               case 'v': /* verbose */
+                       verbose++;
+                       break;
+
+               case 'H': /* help */
+                       print_help();
+                       exit(STATE_UNKNOWN);
+                       break;
+
                default:
                        return ERROR;
                        break;
@@ -229,4 +213,61 @@ int process_arguments(int argc, char **argv){
                return ERROR;
 
        return OK;
-        }
+}
+
+void
+print_help(void)
+{
+       print_revision(progname, revision);
+       printf(COPYRIGHT, copyright, email);
+
+       printf("%s\n", _("Host/Service Cluster Plugin for Nagios 2"));
+
+       print_usage();
+
+
+       printf("%s\n", _("Options:"));
+       printf (" %s\n", "-s, --service");
+       printf ("    %s\n", _("Check service cluster status"));
+       printf (" %s\n", "-h, --host");
+       printf ("    %s\n", _("Check host cluster status"));
+       printf (" %s\n", "-l, --label=STRING");
+       printf ("    %s\n", _("Optional prepended text output (i.e. \"Host cluster\")"));
+       printf (" %s\n", "-w, --warning=THRESHOLD");
+       printf ("    %s\n", _("Specifies the range of hosts or services in cluster that must be in a"));
+       printf ("    %s\n", _("non-OK state in order to return a WARNING status level"));
+       printf (" %s\n", "-c, --critical=THRESHOLD");
+       printf ("    %s\n", _("Specifies the range of hosts or services in cluster that must be in a"));
+       printf ("    %s\n", _(" non-OK state in order to return a CRITICAL status level"));
+       printf (" %s\n", "-d, --data=LIST");
+       printf ("    %s\n", _("The status codes of the hosts or services in the cluster, separated by"));
+       printf ("    %s\n", _("commas"));
+
+       printf(_(UT_VERBOSE));
+
+       printf("\n");
+       printf("%s\n", _("Notes:"));
+       printf(" %s\n", _("See:"));
+       printf(" %s\n", _("http://nagiosplug.sourceforge.net/developer-guidelines.html#THRESHOLDFORMAT"));
+       printf(" %s\n", _("for THRESHOLD format and examples."));
+
+       printf(_(UT_SUPPORT));
+       printf("\n");
+}
+
+
+void
+print_usage(void)
+{
+
+       printf("\n");
+       printf(_("Usage:"));
+       printf(" %s (-s | -h) -d val1[,val2,...,valn] [-l label]\n", progname);
+       printf("[-w threshold] [-c threshold] [-v] [--help]\n");
+       printf("\n");
+
+}
+
+#if 0
+#endif
+