Clarify that only one action is run per invocation.

The old wording could be interpreted as allowing several actions to be
executed in parallel.

Close #115.

Remove some tabs

t/02_txn_time.t: Allow some time slack

t/02_disk_space.t: Further relax device name check

In the wild (Debian sbuild) a CP output of "FS sid-pgdg-amd64-sbuild
mounted on /" was observed. Stop requiring the device name to start with
/ or -, and go with .* instead.

txn_idle: Document that --includeuser can be used to work around superuser checks

Close #81.

connection: Make all errors including timeout from psql CRITICAL

UNKNOWN is not so much useful in the context of basic connection checks.
(The result remains UNKNOWN in case version() returns something fishy.)

Close #100.

Update query_time docs to match actual code

Close #63.

hot_standby_delay: Check server version instead of psql version for features

In passing, make check_postgres.pl executable

Close #74.

Add changelog for c09e7840ed82db6274116ce9b83fbdbc72f940c8.

adjusting check_txn_idle filter by state_change column

t/02_replicate_row.t: wait for the forks, and sleep a bit longer

Hopefully this will make the test more reliable.

t/02_replicate_row.t: Allow for some variation in the replication time

Update MANIFEST

Convert README to Markdown

Fix same_schema on 8.4

Do some whitespace cleanup

... and add a changelog entry for the same_schema fixes by Glyn.

Fix "--filter" logic to honour regular expressions

As per the manual:
"To exclude objects of a certain type by a regular expression against their name, use "noname=regex"."

However the actual check in the script would only filter on an exact match; switched to regex.

Fix username field in check_txn_idle; broken when adding support for objects owned by group roles in commit d55287f1064c615ce2b0eb3cc3e2f6261f8070c4

same_schema: constraint unit test include table name + reinstate sequence last_value check.

Fixes for changes made to same_schema in commit f8145cd902b2c1e01bc211bbfc04737c5fee79f9

  * Amend constraint unit tests in t/02_same_schema.t to take into account we now return the table name
  * Reinstate checking of a sequences last_val, and instead allow users of asynchronous replication to override with '--assume-async' option

Fix same_schema check so it considers left and right operands when comparing operators.

Fix trigger check in check_same_schema to ignore deferrable unique constraint triggers

These triggers have a name containting the triggers oid which mismatches on logical replicas; so to exclude these just ensure tgconstrindid is zero.

Fix typo: missing comma in commit 2e07159c2ecca1bd42950121ccb2f6fad762e30a

Fix same schema to treat pg_constraint.confmatchtype = 'u' = 's'

This relates to a change in the character representing a simple match
on a foreign key stored in the confmatchtype field changing from 'u'
to 's' between postgresql 9.2 and 9.3. Ref:

http://www.postgresql.org/docs/9.2/static/catalog-pg-constraint.html
http://www.postgresql.org/docs/9.3/static/catalog-pg-constraint.html

Fix USERWHERECLAUSE broken when changing pg_user references to pg_role

Modify checks that return or rely on a user name to reference pg_roles rather than pg_user.

This resolves issues where objects owned by a group role are missing in various checks (check_same_schema, check_commitratio, check_hitratio), or detail is missing (check_database_size), or results not filtered as intended (check_last_vacuum_analyze, check_relation_size).

Conflicts:
check_postgres.pl

Fix same_schema to ignore some non-logical schema based values:

operators   - Use text descriptions for result type and operands; ignore related oids
triggers    - Include table name in key to prevent false positive when another table has a trigger with the same name.  Use textual trigger definition; ignore tgqual represe
functions   - Use text function definition; ignore oid based types and defaults.
constraints - Include table name in key to prevent false positive when another table has a constraint with the same name. Use text constraint definition; ignore oid based values.
sequence    - Ignore last_value; I'm confused as to why we'd want to know this on a schema check.  But if required could be added back in with an optional filter instead.
indexes     - Ignore reltablespace oid; we check the tablespace name anyway. Ignore indkey attnum which can differ when tables have been altered differently but are otherwis

Replace $dbh->tables() with selectall_arrayref()

$dbh->tables() seems to be throwing errors in travis. Replace by
a standard catalog query.

t/02_commitratio.t ......... DBD::Pg::db tables failed: no statement executing at t/CP_Testing.pm line 825.
Looks like your test exited with 255 before it could output anything.

Add travis CI integration script to run the check_postgres testsuite on travis-ci.com

Minimize number of errors in the server log during testing

Speed up testing by disabling fsync

Fix t/02_relation_size.t for 8.4 by skipping indexes_size

Add changelog entry for check_replication_slots

Skip replication slot tests before 9.4

(Refers to #109)

Add check_replication_slots check to check the delay on any replication slots.

"Delay" is measured as size of transaction logs retained for the slot e.g:
    check_postgres_replication_slots -db=TEST -H=192.168.0.106 -warning=32M -critical=64M

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-streaming) 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://github.com/jeffjanes/check_postgres

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

Conflicts:
check_postgres.pl

Merge branch 'query-time-show-query' of https://github.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 github.com:terrorobe/check_postgres

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

Quick TODO item

Start 2.21.1 notes