Fix "values are not the same" error in replicate_row when repinfo value is a '0'

When check_replicate_row tested for valid values before starting the check, if the actual value returned was 0 it would evaluate to false, resulting in the error:
"Cannot test replication: values are not the same"

This is resolved by simply checking if the value is defined before assigning it's value a blank string instead.

Remove four more /o flags

removed 'o' modifier from regular expressions

http://stackoverflow.com/questions/550258/does-the-o-modifier-for-perl-regular-expressions-still-provide-any-benefit

Keep using pg_relation_size on 8.4 and earlier

pg_indexes_size doesn't exist either on 8.4, but that action is new, so
let's not care for now.

Also fix t/05_docs.t for the newly added actions.

txn_idle: if seconds are -0, use 0

Close: #75

Change table_size to use pg_table_size()

Change table_size to use pg_table_size(), i.e. to include the TOAST
table size in the numbers reported. Add new actions indexes_size and
total_relation_size, using the respective pg_indexes_size() and
pg_total_relation_size() functions. All size checks will now also check
materialized views where applicable.

Close: #32, #83

Add changelog entry for Correct extra space in perfdata

Close #80

Correct extra space in perfdata

The extra space result in two spaces between the metrics and its incorrectly parse by centreon.

Add changelog entry for last commit, and close #72

Don't fail when query contains 'disabled' word

If a query contains the disabled word (.e.g. select 1 as disabled) "check_postgres.pl --action txn_idle" exits with

```UNKNOWN: No queries - is stats_command_string or track_activities off?```

If you set track_activities to off, the status field contains 'disabled' (at least on 9.3)

Add changelog entries to close #99 and #106

Call psql with option -X

Starting in PostgreSQL 9.6, psql -c no longer implies -X, so it has to
be added explicitly in order to avoid customizations in .psqlrc
interfering with the output parsing.

Run "make html"

"make html" had not been run for some time, do that now.

Merge pull request #106 from mhagander/postgresql.org_https

Update URLs to www.postgresql.org to be https

Update URLs to www.postgresql.org to be https

The main PostgreSQL website is now https only. The versions.rss check
still works by following the redirect, but it's more efficient to hit
the https URL directly.

In passing, also update all the direct links in the documentation to be
https.

Add new argument, --audit-file-dir, to allow control of which directory the same-schema action will use to store its files.

Add some minor casting to help with the bloat query.

t/02_disk_space.t: relax filesystem check

In https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=790231, Daniel
Schepler reports pbuilder (a throwaway-minimal-chroot build wrapper for
Debian packages) to setup the filesystem name to be shown as "-". Allow
this in the testsuite.

Add test for --action=wal_files --lsfunc

Allow a wrapper function for wal_files and archive_ready as a non-superuser.

The wrapper function around pg_ls_dir() must hardcode the pg_xlog or
pgxlog/archive_status path, and should take no arguments. It must be
created as a superuser with SECURITY DEFINER.

Fix hot_standby_delay against a log-shipping (non-) replica.

set_translations.pl: Write output for all known languages

set_translations.pl only wrote output for languages seen in .po files,
dropping other previously known languages.

Also, fix output to honor the current indentation style (no tabs). In
order to change the current formatting as little as possible, set the
minimal output key output width to 14, and no not append a space after
'key' ('checkcluster-nomrtg' is otherwise too long).

Remove duplicated 'es' section

... and note author of last commit in changelog

Add complete spanish (es) message translations

Signatures for version 2.22.0

Let's release

No tabs please

Bump version to 2.22.0

Placeholders for French translation strings

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

Spelling

Rearrange recent changes in rough priority order

Fix txn_time regression test for 9.0/9.1

Newer versions will show the last or current query here, older versions
will just show "<IDLE> in transaction" if there is currently no
query running.

Fix t/02_sequence.t for PG 9.0/1

Update release notes a bit

Version bump.

Remove tabs, cleanup whitespace

Bump copyright

Fix issues with latest changes to check_sequence and run_command:

check_sequence - Split big UNION of sequence selects into chunks to avoid overrunning system argument size

run_command - If we have specified a "target" database, skip all others in global targetdb list

Change the way tables are quoted in replicate_row.

Change the way tables are quoted in replicate_row to allow for tables with schema name.

Add tests for "serial" and "smallserial" columns

The code was already there, but not covered by tests

Query all sequences per DB in parallel for action=sequence

action=sequence used to open a new session for every sequence checked,
which could be very slow. Fix by creating a single SQL statement using
UNION ALL to query all sequences at once.

Allow optional space before 'QUERY PLAN'

Tweak test, as the output no longer ends where it used to

Use Data::Dumper when showing full output on a failed parse, as this is more often than not a complex structure.

Grrr...account for trailing whitespace in psql table output

Thanks a lot, Postgres. Now featuring whitespace in psql output that has never been there before.

Merge dbname2, host2, etc. into the new array method.

Quote values in error message when replicate_row fails

Fix bloat check to use correct SQL depending on the server version.

Previously it used the psql version which might differ when multiple
versions are installed locally, or when querying remote servers.

The hot_standby_delay check is also affected, but more involved to fix,
so leave that for later.

Adrian Vondendriesch

Show OS on failure-to-parse errors

When we fail to parse the psql output, show the entire output string rather than just the failing line, to better aid in debugging as recently seen on the list.

Tweak testing output

Improved multi-slave support in replicate_row.

Merge branch 'space_error' of https://.com/jeffjanes/check_postgres

Merge branch 'master' of https://.com/terrorobe/check_postgres

Conflicts:
check_postgres.pl

Merge branch 'query-time-show-query' of https://.com/petere/check_postgres

Declare POD encoding to be utf8

Quick stub to show port when testing. Always 5432 for now (with a custom socket dir)

Add xact timestamp support to hot_standby_delay

Allow the hot_standby_delay check to accept xlog byte position or
timestamp lag intervals as thresholds, or even both at the same time.

Show actual long-running query in query_time output

fix & extend hot_standby_delay documentation

This should also account for all places where the refactoring of
the numbered dbparameters (host2, port2, etc.) was referenced in the
documentation. Drop the TODO entry.

Merge branch 'master' of .com:terrorobe/check_postgres

Don't swallow space before the -c flag when reporting errors

Quick TODO item

Start 2.21.1 notes

Add an ORDER BY to the slony_status check to account for cases in which
there is more than one node in replication; we want to report on the
most lagged server.

'no warnings' makes Perl::Critic cry. Try a different way.

--host2 and friends no longer work.

Eliminate some "Use of uninitialized value" warnings.

Get rid of number opts in docs.

`curl` does not like `=` between an option and its value.

This gets the `t/02_new_version_cp.t` to pass on systems that use `curl`.

Signatures for 2.21.0

Whitespace

Remove the no warnings hack.

Version bump, better binmode

Spelling tweaks

Version bump

Skip a few more files in the manifest

Spelling fixes

Typo

Perl::Critic insipred fixes

Minor translation fixes

Document testsuite, notably the DBD::Pg version required

Fix regex for cp_version testing.

Bump version to 2.20.2

Adding a jstkind to the dummy pgagent schema, so the recent changes that reference it can work.

t/02_relation_size.t: Sleep 0.5s for the MRTG test

Sleeping 1s created a race condition because we also check after 1s, so
it was only luck whether the result of that check is "1" or "2"

Fix same_schema check on 9.3 by ignoring relminmxid differences in pg_class

t/02_slony_status.t: Fix to work with 9.3

The error was "schema does not exist" until 9.2, and is now "table does
not exist", so just look for "does not exist".

Setting SKIP_NETWORK_TESTS will skip the new_version tests

Build daemons for Debian packages might not have network access, so
provide a way to disable the tests depeding on networking

t/02_last_vacuum.t: Reorder commit calls to make test more stable

Install man page in section 1

Peter Eisentraut, bug 53,  issue 26

Add a changelog entry for the locks fix

Skipping disk_space tests if df doesn't work

df might fail in chroot environments, e.g. on build daemons where
check-postgres packages are built

Sort lock types so the output ordering is predicatable

Credit recent change.

Update check_postgres.pl

Fixing issue with SQL steps in check_pgagent_jobs for jobs which perform deletes, these return non-0 yet are still successful.
Behaviour should now be that <>0 on batch is a failure, but only -1 is a failure for sql steps.

README: Fix two typos

t/02_settings_checksum.t: Accept hex digits

Previously, this check only worked by chance if the returned checksum
started with a plain digit.