Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better check & document required permissions #166

Open
HOSTED-POWER opened this issue Oct 9, 2019 · 0 comments
Open

Better check & document required permissions #166

HOSTED-POWER opened this issue Oct 9, 2019 · 0 comments

Comments

@HOSTED-POWER
Copy link

HOSTED-POWER commented Oct 9, 2019

Hi,

We tried the check_postgres for example for checking long running queries (--action query_time). However when we didn't have enough permissions it just outputted:

POSTGRES_QUERY_TIME OK: DB "postgres" (host:127.0.0.1) longest query: 0s

Now with sufficient permissions it outputs:

POSTGRES_QUERY_TIME WARNING: DB "postgres" (host:127.0.0.1) longest query: 16s

I would expect the first one to simply output UNKNOWN because of lacking permissions :)

Atm we grant:

grant pg_read_all_settings, pg_read_all_stats, pg_stat_scan_tables to xxx;

I suppose this should be a bit better documented and for sure output error if it lacks permissions for certain actions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

0 participants