Skip to content

Conversation

@kakuiho
Copy link

@kakuiho kakuiho commented Dec 9, 2024

see also #1100

@kakuiho
Copy link
Author

kakuiho commented Feb 5, 2025

@khiemdoan Could you please merge this PR?

@khiemdoan
Copy link
Contributor

Hi @kakuiho,
So sorry, I don't have permission to merge any PR.

SELECT * FROM get_pg_stat_statements();
GRANT SELECT ON postgres_exporter.pg_stat_statements TO postgres_exporter;
GRANT ALL ON FUNCTION pg_catalog.pg_ls_waldir() TO postgres_exporter;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I always hesitate to recommend someone give ALL permissions. I think this only needs EXECUTE. Can you confirm and update?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran on this comment by chance and did it on my side as I had the error, and it did clear the error from the console when granting only EXECUTE. So I guess it does work 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants