From notifications at github.com Fri Aug 1 00:01:54 2014 From: notifications at github.com (Rob Thomas) Date: Thu, 31 Jul 2014 15:01:54 -0700 Subject: Update spec file (#1283) In-Reply-To: References: Message-ID: Changed the summary to Summary: Collection of monitoring plugins for Sensu/Icinga/Nagios and other Nagios compatible programs How's that? -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1283#issuecomment-50824893 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Fri Aug 1 23:05:40 2014 From: notifications at github.com (Mark A. Ziesemer) Date: Fri, 01 Aug 2014 14:05:40 -0700 Subject: Add support for showing name (DNS) resolution in the `check_ping` plugin output (STDOUT). (#1284) Message-ID: Add support for showing name (DNS) resolution in the `check_ping` plugin output (STDOUT). As written (alternate suggestions welcome), after being enabled with `-s` or `--show-resolution`, the output of a sample command (including the shorthand `-s` combined with the standard `-H`): check_ping -sH someHost -w 3000.0,80% -c 5000.0,100% -p 5 ... will be presented as: PING OK - Packet loss = 0%, RTA = 0.63 ms - someHost.myDomain.com (10.1.2.3) .. instead of the typical: PING OK - Packet loss = 0%, RTA = 0.63 ms In my monitoring configurations, DNS names are used whenever possible - to ensure that any failures in DNS resolution (either of the DNS service as a whole, or just for a given entry) are also reflected in the status of a host check. The reduces the cost of recurring maintenance, as the monitoring configurations don't also need to be updated every time there is something as simple as an IP change. Having a local DNS cache on the server (e.g. `dnsmasq`) makes any added performance expense trivial. The default `ping` output already includes the following as the first line (as of version `iputils-sss20071127`): PING someHost (10.1.2.3) 56(84) bytes of data. This is potentially useful information, and this change simply makes the resolved hostname (potentially different than an input hostname) and resolved IP address available for display on the plugin output. You can merge this Pull Request by running: git pull https://github.com/ziesemer/monitoring-plugins master Or you can view, comment on it, or merge it online at: https://github.com/monitoring-plugins/monitoring-plugins/pull/1284 -- Commit Summary -- * Update check_ping.c * Update check_ping.c -- File Changes -- M plugins/check_ping.c (75) -- Patch Links -- https://github.com/monitoring-plugins/monitoring-plugins/pull/1284.patch https://github.com/monitoring-plugins/monitoring-plugins/pull/1284.diff -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1284 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Tue Aug 12 00:08:48 2014 From: notifications at github.com (waja) Date: Mon, 11 Aug 2014 15:08:48 -0700 Subject: check_ntp: pointer-from-integer (#1285) Message-ID: https://qa.debian.org/bls/packages/m/monitoring-plugins.html indicates that check_ntp had a sloppy commit in the recent past: pointer-from-integer check_ntp.c:619 - https://github.com/monitoring-plugins/monitoring-plugins/blob/master/plugins/check_ntp.c#L619 This commes with https://github.com/monitoring-plugins/monitoring-plugins/commit/a04df3e1b67dc5eab3adc202cc89901f801cdeaa -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/issues/1285 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Sat Aug 23 20:12:46 2014 From: notifications at github.com (waja) Date: Sat, 23 Aug 2014 11:12:46 -0700 Subject: check_http: chunked Transfer-Encoding (#1286) Message-ID: Cloned from https://github.com/nagios-plugins/nagios-plugins/pull/54 provided by @koentje You can merge this Pull Request by running: git pull https://github.com/waja/monitoring-plugins chunked_http_transfer Or you can view, comment on it, or merge it online at: https://github.com/monitoring-plugins/monitoring-plugins/pull/1286 -- Commit Summary -- * Make test/check_http.t reuse sockets * Add testcase for HTTP chunked encoding * plugins/check_http: chunked Transfer-Encoding * plugins/check_http: def. function conform style * plugins/check_http: Improve readability * plugins/check_http: Improve readability some more -- File Changes -- M plugins/check_http.c (85) M plugins/tests/check_http.t (24) -- Patch Links -- https://github.com/monitoring-plugins/monitoring-plugins/pull/1286.patch https://github.com/monitoring-plugins/monitoring-plugins/pull/1286.diff -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1286 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Tue Aug 26 14:49:59 2014 From: notifications at github.com (Yannick Charton) Date: Tue, 26 Aug 2014 05:49:59 -0700 Subject: Added particular ps command for HP-UX (check_procs) (#1287) Message-ID: Hi, check_procs doesn't properly work on HP-UX systems. This change makes it working with almost all options. Only rss values are not provided. Regards, Yannick You can merge this Pull Request by running: git pull https://github.com/Tontonitch/monitoring-plugins master Or you can view, comment on it, or merge it online at: https://github.com/monitoring-plugins/monitoring-plugins/pull/1287 -- Commit Summary -- * Added particular ps command for HP-UX (check_procs) -- File Changes -- M configure.ac (14) -- Patch Links -- https://github.com/monitoring-plugins/monitoring-plugins/pull/1287.patch https://github.com/monitoring-plugins/monitoring-plugins/pull/1287.diff -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1287 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Tue Aug 26 16:08:10 2014 From: notifications at github.com (Yannick Charton) Date: Tue, 26 Aug 2014 07:08:10 -0700 Subject: Added particular ps command for HP-UX (check_procs) (#1287) In-Reply-To: References: Message-ID: Typo problem in the commit, closing and will reopen -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1287#issuecomment-53425324 -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Tue Aug 26 16:17:41 2014 From: notifications at github.com (Yannick Charton) Date: Tue, 26 Aug 2014 07:17:41 -0700 Subject: check_procs: added working HP-UX support (#1288) Message-ID: Hi, check_procs doesn't properly work on HP-UX systems. This change makes it working with almost all options. Only rss values are not provided. Regards, Yannick You can merge this Pull Request by running: git pull https://github.com/Tontonitch/monitoring-plugins master Or you can view, comment on it, or merge it online at: https://github.com/monitoring-plugins/monitoring-plugins/pull/1288 -- Commit Summary -- * Added particular ps command for HP-UX (check_procs) * Added particular ps command for HP-UX (check_procs) - typo fix -- File Changes -- M configure.ac (14) -- Patch Links -- https://github.com/monitoring-plugins/monitoring-plugins/pull/1288.patch https://github.com/monitoring-plugins/monitoring-plugins/pull/1288.diff -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1288 -------------- next part -------------- An HTML attachment was scrubbed... URL: