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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants