Opened 3 years ago

Closed 6 months ago

#404 closed defect (fixed)

CheckEventLog returns bad syntax

Reported by: sr_dev Owned by: mickem
Priority: 1 Milestone: 0.4.1
Component: CheckEventlog Version: 0.3.8
Severity: Bugs Keywords: CheckEventLog
Cc:

Description

nsclient 0.3.8:

Query:

-c CheckEventLog -a file=application filter+eventType==error MaxWarn=100 MaxCrit=1000 filter=id=xxxx truncate=800 "filter+generated=<24h" unique descriptions

Returns:

success: xxxx: (93)|'eventlog'=93;100;1000;

When altered filter-id it returns the expected:

Eventlog check ok|'eventlog'=0;100;1000;

Change History (7)

comment:1 follow-up: Changed 3 years ago by mickem

Not sure I understand what the problem is.

Eventlog check ok|'eventlog'=0;100;1000; 

means "no problems where found"

success: xxxx: (93)|'eventlog'=93;100;1000; 

Means something was found.

And if you change the filter you will mot likely get different results.

Michael Medin

comment:2 in reply to: ↑ 1 Changed 3 years ago by sr_dev

Replying to mickem:

Not sure I understand what the problem is.

Eventlog check ok|'eventlog'=0;100;1000; 

means "no problems where found"

success: xxxx: (93)|'eventlog'=93;100;1000; 

The problem is that the syntax is confusing because it refers to the
technical filter result, not the real result. For example the query above which returns following to Nagios:

Mysoftware, CRITICAL, success: My app is out of diskspace.

The examples in the Nagios API docs use:

DISK OK - free space: / 3326 MB (56%); | /=2643MB;5948;5958;0;5968

which in Nagios translates to

Disk util, OK, OK: C: 7.81G, D: 200M

See my point? :)

David Lundberg

comment:3 Changed 3 years ago by mickem

  • Owner set to mickem

Not really...

The "real result" is returned in the "result code" which Nagios parses into a state, you would want that result to be repeated in the message as well?

That could easily be achieved via scripting.

But I shall see might be something to consider in the future...

Michael Medin

comment:4 Changed 3 years ago by mickem

  • Milestone set to 0.5.0

comment:5 Changed 15 months ago by mickem

  • Component changed from None to CheckEventlog

comment:6 Changed 15 months ago by mickem

  • Milestone changed from 0.5.0 to future

comment:7 Changed 6 months ago by mickem

  • Milestone changed from future to 0.4.1
  • Resolution set to fixed
  • Status changed from new to closed

can be configured in 0.4.1

Note: See TracTickets for help on using tickets.