Get rid of "uninitialized value" warnings.

Credit recent change

check_txn_idle: fix query_time --excludeuser query

Parenthesis are required:

"a OR b AND usename <> 'foo'" -> "(a OR b) AND usename <> 'foo'"

Fixes at least "query_time --includeuser/--excludeuser".

Credit recent change

Fix userclause handling in check_txn_idle

For databases > Postgrs 9.1, irrelevant records are discarded
with two ORed conditions. This makes the optional userclause
associate with the right hand side of the condition. The expression
needs to be grouped to get the desired effect regardless of the
evaluation of the previous condition.

Bump year to 2013 and some minor HTML adjustments.

Make sure hot_standby_delay doesn't return negative values

Before this patch, the hot_standby_delay action could return negative values.
To minimize the risk, we first query the slave, then the master. And to make
sure, we never get negative values, we set them to zero if we have a negative
value.

Report and ideas from Filip Rembiałkowski.

Credit recent change

Make sure SQL2 is defined, per github issue 16

Postgres 9.2 compatibility

check_txn_idle and check_backends (and the backends test) have been
updated to use the new pg_stat_activity.  And check_disk_space now uses
pg_tablespace_location().

Fix disk_space docs

Comments from Lance Raymond.

Begin support for new 9.2 pg_stat_activity.
This is just the groundwork: it will not work, as we have to use the new 'state'
column instead of parsing the text of current_query.

Credits

For the locks action, always output all databases with values of zero as needed. Per request from the mailing list. Also adjust the help messages to use : as a separator, not ;

Merge remote branch 'f0rk/f0rk/empty_tables_bloat'

Force to port 5432 for funky systems.

Bail out early if the test server is completely unreachable

Note that locks args can contain colons or semicolons or anything else: make the docs use colons.

Add label for the recent next.

Version bump

Spelling fixes.

Minor cleanups.

Spelling fixes

Cache our specific sequence information during same_schema.
This greatly speeds things up for large schemas!

Merge remote branch 'bucardo/master' into f0rk/empty_tables_bloat

Restore totalwastedbytes

Merge remote branch 'f0rk/f0rk/ignore_temp_sequences'

Cache the pg_typmod lookups.

Make check_bloat gracefully deal with empty tables

Ignore sequences in the temporary namespace

Better --dbservice docs, as pointed out by Jason Ryan on the mailing list.

Merge branch 'master' of bucardo.org:/var/lib/git/check_postgres

Remove odd whitespace junk.

Fix check_postgres_hitratio documentation

check_postgres_hitratio was wrong called
check_postgres_database_hitratio in the documentation.

Allow txn_idle to work again for < 8.3 servers by switching to query_time.

Better detection of stats_command_string being off.
Start work to allow txn_idle on old systems.

Cleaner "ERROR" output

Fix French UTF-8 character.

Allow spaces within items within same_schema action.
Bump version to 2.20.0

Note recent change

Merge branch 'master' of bucardo.org:/var/lib/git/check_postgres

Make sure STDOUT is using utf8. Per report from Emmanuel Lesouef

Update the french translation

Fix repository entry in META.yml

just a trailling / and a s/https/git/ change

Allows PGBINDIR in check_postgresrc

This is the same approach as it is with PSQL variable (the $NO_PSQL_OPTION
variable har-coded *MUST* be set to 0 to use them).

PGBINDIR can be set in ENV, hard-coded in check_postgres.pl or set via
command-line argument.
PGCONTROLDATA and PSQL can still be used but should be deprecated.

the logic is if NO_PSQL_OPTION=1 :
 * that PGBINDIR can be set via environment variable, but not via config
file or command-line argument.
 * that PSQL can not be set explicitly, but derived from PGBINDIR

Fix the createlang during init

There was a copy-n-paste error, only affecting < 8.1 .

Revert "Minor change in bloat test"

This reverts commit 7788d081ab5ed74d53f699bc7333e00f2bb59d9a.

GSM: Data::Dumper is handy for when you do want to Dump something.

Note the symlink for generic tarball

Update History for 2.20.0

Add check for pgagent jobs (David E. Wheeler)

Minor change in bloat test

The test was loading Data::Dumepr but not using it.

Add `pgagent_jobs` test.

From: "David E. Wheeler" <[email protected]>

This patch adds support for checking for failed pgAgent jobs within a specified
period of time. You can specify either --critical or --warning as a period of
time, and it will report on failures within that period of time previous to the
current time. Job failures are determined by a non-0 status in a job step
record.

Using this test obviously requiers that the pgAgent schema be installed. I've
also included a bunch of unit tests to make sure it works the way I would expect
(the test will create a schema for testing) and documentation.

As part of this, I've introduced the `any_warning` argument to
`validate_range()`. The `pgagent_jobs` test does not care if you specify a
warning value greater than the critical value (indeed, I expect that if one used
both at all, the warning would be much longer). So this new argument prevents
the `range-warnbigtime` or `range-warnbigsize` failures from being triggered.

Cedric: I sorted the POD and added the action_info so that t/05_docs.t is ok.
I also built and push the new .html

Add link

More tweaks to the release process.

Note need to commit before the tagging.

Ready for 2.19.0 release

Perl::Critic inspired changes

Omit Variables::ProhibitUselessInitialization as it has an internal error.

Try to wrap failing Perl::Critic calls in an eval. Does not work.

Clear temp schema

Remove tabs

Tweaks for better "fakeschema" in the tests.

Have MRTG show a set message, for times when we end up there indirectly.

Spell check tweaks

Translation cleanups

Typo in SQL

Make sure we do not force cluster ids to an int, else we get exponential notation.

Add some debugging to the perlcritic test

Getting tired of these !@#$!@#%! tabs in this !~@#$!@#$ code

More tabs

Remove tabs

Bump to 2012

Ignore the MYMETA auto-generated files

Make version 2.19.0 everywhere

Remove version 2.18.1 - going right to 2.19

Note bugzilla items, per recent email.

Fix xlog formula: ff000000 not ffffffff
See http://eulerto.blogspot.com/2011/11/understanding-wal-nomenclature.html

Revert bug 92 tweak, per point from Robert Treat.
No more late night coding for me. #easilybrokenpromises

Exclude IIT from txn_time

Merge branch 'master' of bucardo.org:/var/lib/git/check_postgres

Note latest fix.

Change join to pg_user to a left join, as the database may be owned by a role.
Thanks to Emmanuel Lesouef for the bug report and help in tracking this down.
There are probably other incorrect inner joins to pg_user in the code.

Update history

Add MRTG to test for settings_checksum

Add a test for checkpoint and --assume-standby-mode

Add cluster_id test

for automatic testing too.

Add an --assume-prod option

This is based on --assume-standby-mode. Reduce the option name per suggestion
from Greg (but I kept the original one for standby mode).

The option is only used in check_postgres_checkpoint and allows to confirm or
emit a critical if the server is not in the expected mode.

Note: this can be used in other places, and maybe improved (to reduce the
number og open_controldata calls)

TODO/FIXME:
* I found that --assume-p or --assume-s are viewed by GetOpt like the longer
version of the option, a bug ?
* The original code to call pg_controldata does not work in French (because of
regex/locale). Why not use LANG=C in those checks where there is NO point to
use locale and error prone regex ?

Reduce code duplicates around pg_controldata

Use the open_controldata where pg_controldata was used previously.
Also split the code for make_sure_standby_mode to reduce code for the future
option make_sure_prod.

Add a check cluster_id

This check is responsible to confirm that the Database System Identifier found
by pg_controldata is the one expected.
warning and critical allowed (like check_postgres_checksum) and must be run on
PostgreSQL server (like check_postgres_checkpoint)

While here, I created a new function open_controldata which can be used in
other places where pg_controldata is used.

Note last changes

Create a man 3 entry

We already have check_postgres.pl listed as an EXE, so prevent Makefile from putting it into the Perl lib path as well.

Fix typo.

Fix uninitialized string warnings when host or port not specified (eg using dbservice)

Redo latest orderby tweaks

Credit last patch

Show dbservice if used at start of same_schema header.

Credit bloat change.

Get version 2.18.1 ready

Use the full path when getting sequence information for same_schema.
Thanks to Cindy Wise for the bug report.

Show bloat when indexes are high too