[check_postgres] [PATCH] Add `pgagent_jobs` test.

[Cédric Villemain]

Le 19 janvier 2012 23:02, David E. Wheeler
<david.wheeler at pgexperts.com> a écrit :
> On Jan 19, 2012, at 1:58 PM, Cédric Villemain wrote:
>
>> emit critical if > 2 hours (or < 0)
>> It breaks at least "query_runtime" and "last*analyze|vacuum" checks
>> for time range . but is ok for others time range check, I didn't check
>> others checks.
>>
>> For me it is an improvement: better nagios integration, easier
>> documentation for check_postgres (less confusion on what -w -c mean in
>> this or this check).
>>
>> I won't argue very strongly to break backward compatibility.
>> We can decide that some checks are not clean and we accept the old
>> syntax for them (but not your new one :p).
>
> Yeah, that’s what I was thinking: If there is no :, fall back on the old parser.
>
>> How to handle change in !clean checks is probably going to make it in
>> its own thread.
>
> Given that it might take some effort, I think I might like to push my change now, and then refactor for your changes when they’re done (dropping any_warning at that time). Thoughts?

ok.

I am looking at it right now.
-- 
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation