Turn the exec in validate_db_connection.pp around

The Exec "validate postgres connection for ${database_host}/${database_name}" is always executed and returns a notice every time it is executed successfully. A node which uses this, will never turn ' green' in puppet-dashboard.

I do not like that, so i changed the exec. It does the validation now in within the 'unless' statement . If that returns something else then 0 (zero), the command '/bin/false' is executed. /bin/false returns a 1 (one), what will cause the Exec to fail.

Effectively the check does exact the same as before, but now nothing is logged, except when it fails.