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

Update french translation.

Fix query_time action when using --dbservice option

When using --dbservice option, $db->{dbname} is not set. So the $whodunit
variable initialization should first check if the $db->{dbname} is set.

Actually, we only initialize it for the MRTG output as it is not important in
the Nagios output.

Fix query time action

The query_time action works before 8.1 if the query doesn't include the
xact_start column. But we need this column for the txn_time action. So, I
changed the query so that the query_time can work with 8.1 and upwards, and
that the txn_time works with 8.3 and upwards.

Allow the check of multiple db with connection action

We still stop at the first error.

Per request from Aziz Boultabi.

PGP sig for 2.18.0

Final tweaks for 2.18.0

Signature for 2.18.0

Spellcheck updates

Message/translation cleanups.

Typo.

Fix some "make test" issues

* backends test issue
  Critical and warning values were wrong for the negative number check.
  And the output message for the --include check was wrong too.

* check_replicate_row issue
  The UPDATE must be executed on the first server only.

* fsm_pages and fsm_relations test issue
  The version test (ie max_fsm_* not available on 8.4 and later releases) must be done earlier.

* doc test issue
  check_standby_mode rename to make_sure_standby_mode because all check_*
  functions are expected to have documentation, but check_standby_mode is an
  internal function.

* another doc test issue
  pgbouncer_checksum documentation wasn't at the right location.

* drop_schema_if_exists issue
  There was an unexpected return in the middle of the function, and so the
  schema was never dropped.

New french update

Note latest changes. Quick document of same_schema --replace

Fix space in perfdata for hot_standby_delay action.

Don't allow this test unless in RELEASE mode.

Note the special ignore/exclude handling in the bloat routine, per discussion on the mailing list.

New option "replace" to allow replacing the audit file after a run.

More prosrc filtering tweaks.

Don't tell us about the sequence number changing over time!

Support per-version SQL for same_schema action.

Use the entire sub call for the "totaltime" counter for same_schema.

Do not show connection variants for historical database.

Switch around probin munging.

Credit last commit

Fix typo in dbstats action.

Clean up all versions of test database dirs and their sockets.

Keep the fake schema around, as we modify search_path with ALTER USER

Various testing fixes.

The database name is not always available (e.g. check_checkpoint)

Tweak backend tests to account for new output

Remove a tab that snuck in.

Better tests for non-superuser in txn_idle and friends.
Attempts to fix bug 59.

printf fixes.

Note that txn_idle and query_time require 8.3 (Thomas Guettler)

Be clear that we are checking the pg_listener table. Per suggestion from Thomas Guettler.
Note the failure to work at versions >= 9.0

Doc typo in prepared_txns section.

Credit Thomas Guettler with last commit idea.

Set some defaults for archive_ready and wal_files. Adjustements welcome.
Also make the output for both actions clearer.

Cleanup the History section.

Allow the --schema argument for the slony_status action to take multiple values, and map those back into the targetdbs as needed. Also detect multiple slonys per database if no --schema is given. Per idea from Jehan-Guillaume de Rorthais.

Fix tests for new naming.

Rename test files

Test file tweaks for new output.

Bump to 2.18.0.
Update Changes

Spell check updates.

Complete rewrite of the same_schema action.
Now supports >2 databases for comparison,
as well as "historical" comparisons of the same db over time!

Fix ORDER BY in the last vacuum/analyze action

Before this patch, the "ORDER BY" was on the relname column. This fixes it to
get an ORDER BY on the last time column, which is more interesting
(especially when you have a perflimit).

Fix check_hot_standby_delay perfdata output

There was a missing space in the perfdata output for this action.

Fix the archive_ready action

The .ready files are not in the pg_xlog directory, but in the
pg_xlog/archive_ready one. This patch adds another argument to the
check_wal_files() function so that it could find the subdirectory in which it
has to look for files.

Rename recent tests: 'database' is implicit for most things.
Plus, I didn't want to have to re-line up everything inside $action_info :)

Add the hitratio action

This new action allows a user to track the hit ratio, ie the difference
between cache and disk reads, in each database.

Add the commitratio action

This new action allows a user to track the commit ratio, ie the difference
between commit and rollback, in each database.

Make sure --action overrides the naming trick.

Quick fix for cases when client_port is set to an empty string, per bug report 79 from Fabrice Metge