![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
diff --git a/plugins/check_snmp.c b/plugins/check_snmp.c
index 2bc6024f42bab4ca06475ab69473f3d1fc64e729..bf2102220b06140c61ff5a11f28bfb69b91c2557 100644 (file)
--- a/plugins/check_snmp.c
+++ b/plugins/check_snmp.c
#define MAX_OIDS 8
+/* Longopts only arguments */
+#define L_CALCULATE_RATE CHAR_MAX+1
+#define L_RATE_MULTIPLIER CHAR_MAX+2
+
/* Gobble to string - stop incrementing c when c[0] match one of the
* characters in s */
#define GOBBLE_TOS(c, s) while(c[0]!='\0' && strchr(s, c[0])==NULL) { c++; }
char *output_delim;
char *miblist = NULL;
int needmibs = FALSE;
+int calculate_rate = 0;
+int rate_multiplier = 1;
+state_data *previous_state;
+double previous_value[MAX_OIDS];
int
main (int argc, char **argv)
{
- int i, len, line;
+ int i, len, line, total_oids;
unsigned int bk_count = 0, dq_count = 0;
int iresult = STATE_UNKNOWN;
int result = STATE_UNKNOWN;
char *th_crit=NULL;
char type[8] = "";
output chld_out, chld_err;
+ char *previous_string=NULL;
+ char *ap=NULL;
+ char *state_string=NULL;
+ size_t response_length, current_length, string_length;
+ char *temp_string=NULL;
+ int is_numeric=0;
+ time_t current_time;
+ double temp_double;
+ time_t duration;
+ char *conv = "12345678";
+ int is_counter=0;
setlocale (LC_ALL, "");
bindtextdomain (PACKAGE, LOCALEDIR);
timeout_interval = DEFAULT_TIMEOUT;
retries = DEFAULT_RETRIES;
+ np_init( (char *) progname, argc, argv );
+
/* Parse extra opts if any */
argv=np_extra_opts (&argc, argv, progname);
+ np_set_args(argc, argv);
+
if (process_arguments (argc, argv) == ERROR)
usage4 (_("Could not parse arguments"));
+ if(calculate_rate) {
+ if (!strcmp(label, "SNMP"))
+ label = strdup("SNMP RATE");
+ time(¤t_time);
+ i=0;
+ previous_state = np_state_read();
+ if(previous_state!=NULL) {
+ /* Split colon separated values */
+ previous_string = strdup((char *) previous_state->data);
+ while((ap = strsep(&previous_string, ":")) != NULL) {
+ if(verbose>2)
+ printf("State for %d=%s\n", i, ap);
+ previous_value[i++]=strtod(ap,NULL);
+ }
+ }
+ }
+
/* Populate the thresholds */
th_warn=warning_thresholds;
th_crit=critical_thresholds;
}
for (line=0, i=0; line < chld_out.lines; line++, i++) {
- const char *conv = "%.0f";
+ if(calculate_rate)
+ conv = "%.10g";
+ else
+ conv = "%.0f";
ptr = chld_out.line[line];
oidname = strpcpy (oidname, ptr, delimiter);
/* Clean up type array - Sol10 does not necessarily zero it out */
bzero(type, sizeof(type));
+ is_counter=0;
/* We strip out the datatype indicator for PHBs */
- if (strstr (response, "Gauge: "))
+ if (strstr (response, "Gauge: ")) {
show = strstr (response, "Gauge: ") + 7;
- else if (strstr (response, "Gauge32: "))
+ is_numeric++;
+ }
+ else if (strstr (response, "Gauge32: ")) {
show = strstr (response, "Gauge32: ") + 9;
+ is_numeric++;
+ }
else if (strstr (response, "Counter32: ")) {
show = strstr (response, "Counter32: ") + 11;
- strcpy(type, "c");
+ is_numeric++;
+ is_counter=1;
+ if(!calculate_rate)
+ strcpy(type, "c");
}
else if (strstr (response, "Counter64: ")) {
show = strstr (response, "Counter64: ") + 11;
- strcpy(type, "c");
+ is_numeric++;
+ is_counter=1;
+ if(!calculate_rate)
+ strcpy(type, "c");
}
- else if (strstr (response, "INTEGER: "))
+ else if (strstr (response, "INTEGER: ")) {
show = strstr (response, "INTEGER: ") + 9;
+ is_numeric++;
+ }
else if (strstr (response, "STRING: ")) {
show = strstr (response, "STRING: ") + 8;
conv = "%.10g";
iresult = STATE_DEPENDENT;
- /* Process this block for integer comparisons */
- if (thlds[i]->warning || thlds[i]->critical) {
+ /* Process this block for numeric comparisons */
+ if (is_numeric) {
ptr = strpbrk (show, "0123456789");
if (ptr == NULL)
die (STATE_UNKNOWN,_("No valid data returned"));
response_value[i] = strtod (ptr, NULL);
- iresult = get_status(response_value[i], thlds[i]);
- asprintf (&show, conv, response_value[i]);
+
+ if(calculate_rate) {
+ if (previous_state!=NULL) {
+ duration = current_time-previous_state->time;
+ if(duration<=0)
+ die(STATE_UNKNOWN,_("Time duration between plugin calls is invalid"));
+ temp_double = (response_value[i]-previous_value[i])/duration;
+ /* Simple overflow catcher (same as in rrdtool, rrd_update.c) */
+ if(is_counter) {
+ if(temp_double<(double)0.0)
+ temp_double+=(double)4294967296.0; /* 2^32 */
+ if(temp_double<(double)0.0)
+ temp_double+=(double)18446744069414584320.0; /* 2^64-2^32 */;
+ }
+ iresult = get_status(temp_double, thlds[i]);
+ asprintf (&show, conv, temp_double);
+ }
+ } else {
+ iresult = get_status(response_value[i], thlds[i]);
+ asprintf (&show, conv, response_value[i]);
+ }
}
/* Process this block for string matching */
}
/* Process this block for existence-nonexistence checks */
+ /* TV: Should this be outside of this else block? */
else {
if (eval_method[i] & CRIT_PRESENT)
iresult = STATE_CRITICAL;
result = max_state (result, iresult);
/* Prepend a label for this OID if there is one */
- if (nlabels > (size_t)1 && (size_t)i < nlabels && labels[i] != NULL)
+ if (nlabels >= (size_t)1 && (size_t)i < nlabels && labels[i] != NULL)
asprintf (&outbuff, "%s%s%s %s%s%s", outbuff,
(i == 0) ? " " : output_delim,
labels[i], mark (iresult), show, mark (iresult));
ptr = NULL;
strtod(show, &ptr);
if (ptr > show) {
- strncat(perfstr, oidname, sizeof(perfstr)-strlen(perfstr)-1);
+ if (nlabels >= (size_t)1 && (size_t)i < nlabels && labels[i] != NULL)
+ temp_string=labels[i];
+ else
+ temp_string=oidname;
+ if(calculate_rate)
+ asprintf(&temp_string,"%s-rate",temp_string);
+ strncat(perfstr, temp_string, sizeof(perfstr)-strlen(perfstr)-1);
strncat(perfstr, "=", sizeof(perfstr)-strlen(perfstr)-1);
len = sizeof(perfstr)-strlen(perfstr)-1;
strncat(perfstr, show, len>ptr-show ? ptr-show : len);
strncat(perfstr, " ", sizeof(perfstr)-strlen(perfstr)-1);
}
}
+ total_oids=i;
+
+ /* Save state data, as all data collected now */
+ if(calculate_rate) {
+ string_length=1024;
+ state_string=malloc(string_length);
+ if(state_string==NULL)
+ die(STATE_UNKNOWN, _("Cannot malloc"));
+
+ current_length=0;
+ for(i=0; i<total_oids; i++) {
+ asprintf(&temp_string,"%.0f",response_value[i]);
+ if(temp_string==NULL)
+ die(STATE_UNKNOWN,_("Cannot asprintf()"));
+ response_length = strlen(temp_string);
+ if(current_length+response_length>string_length) {
+ string_length=current_length+1024;
+ state_string=realloc(state_string,string_length);
+ if(state_string==NULL)
+ die(STATE_UNKNOWN, _("Cannot realloc()"));
+ }
+ strcpy(&state_string[current_length],temp_string);
+ current_length=current_length+response_length;
+ state_string[current_length]=':';
+ current_length++;
+ free(temp_string);
+ }
+ state_string[--current_length]='\0';
+ if (verbose > 2)
+ printf("State string=%s\n",state_string);
+
+ /* This is not strictly the same as time now, but any subtle variations will cancel out */
+ np_state_write_string(current_time, state_string );
+ if(previous_state==NULL) {
+ /* Or should this be highest state? */
+ die( STATE_OK, _("No previous data to calculate rate - assume okay" ) );
+ }
+ }
printf ("%s %s -%s %s\n", label, state_text (result), outbuff, perfstr);
if (mult_resp) printf ("%s", mult_resp);
{"authpasswd", required_argument, 0, 'A'},
{"privpasswd", required_argument, 0, 'X'},
{"next", no_argument, 0, 'n'},
+ {"rate", no_argument, 0, L_CALCULATE_RATE},
+ {"rate-multiplier", required_argument, 0, L_RATE_MULTIPLIER},
{0, 0, 0, 0}
};
output_delim = strscpy (output_delim, optarg);
break;
case 'l': /* label */
- label = optarg;
nlabels++;
if (nlabels >= labels_size) {
labels_size += 8;
unitv[nunits - 1] = ptr;
}
break;
-
+ case L_CALCULATE_RATE:
+ if(calculate_rate==0)
+ np_enable_state(NULL, 1);
+ calculate_rate = 1;
+ break;
+ case L_RATE_MULTIPLIER:
+ if(!is_integer(optarg)||(rate_multiplier=atoi(optarg)<=0))
+ usage2(_("Rate multiplier must be a positive integer"),optarg);
+ break;
}
}
printf (" %s\n", _("Warning threshold range(s)"));
printf (" %s\n", "-c, --critical=THRESHOLD(s)");
printf (" %s\n", _("Critical threshold range(s)"));
+ printf (" %s\n", "--rate");
+ printf (" %s\n", _("Enable rate calculation. See 'Rate Calculation' below"));
/* Tests Against Strings */
printf (" %s\n", "-s, --string=STRING");
printf (" %s\n", "-R, --eregi=REGEX");
printf (" %s\n", _("Return OK state (for that OID) if case-insensitive extended REGEX matches"));
printf (" %s\n", "-l, --label=STRING");
- printf (" %s\n", _("Prefix label for output from plugin (default -s 'SNMP')"));
+ printf (" %s\n", _("Prefix label for output from plugin (default -l 'SNMP')"));
/* Output Formatting */
printf (" %s\n", "-u, --units=STRING");
printf (" %s\n", _("- All evaluation methods other than PR, STR, and SUBSTR expect that the value"));
printf (" %s\n", _("returned from the SNMP query is an unsigned integer."));
+ printf("\n");
+ printf("%s\n", _("Rate Calculation:"));
+ printf(" %s\n", _("In many places, SNMP returns counters that are only meaningful when"));
+ printf(" %s\n", _("calculating the counter difference since the last check. check_snmp"));
+ printf(" %s\n", _("saves the last state information in a file so that the rate can be"));
+ printf(" %s\n", _("calculated. Use the --rate option to save state information. On the"));
+ printf(" %s\n", _("first run, there will be no prior state - this will return with OK."));
+ printf(" %s\n", _("The state is uniquely determined by the arguments to the plugin, so"));
+ printf(" %s\n", _("changing the arguments will create a new state file."));
+
printf (UT_SUPPORT);
}