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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(sql-exporter): updated deployments and architecture #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

qfritz
Copy link
Contributor

@qfritz qfritz commented Feb 2, 2024

Last commits to update our tutorials and reference after migration to sql-exporter.

馃檵 Would you be able to check the exporter-deployment.md file please, and complete it if needed? Thanks in advance.

- idx_blks_hit:
usage: "COUNTER"
description: "Number of buffer hits in all indexes on this table"
CREATE ROLE sql_exporter IN ROLE pg_monitor PASSWORD 'hackme' LOGIN; -- You may use a better password
Copy link
Contributor

Choose a reason for hiding this comment

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

We should use prometheus_sql_exporter

Suggested change
CREATE ROLE sql_exporter IN ROLE pg_monitor PASSWORD 'hackme' LOGIN; -- You may use a better password
CREATE ROLE prometheus_sql_exporter IN ROLE pg_monitor PASSWORD 'hackme' LOGIN; -- You may use a better password

- postgres_15_instance_pg_stat_bgwriter
static_configs:
- targets:
pg1: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/postgres'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
pg1: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/postgres'
pg1: 'pg://prometheus_sql_exporter:${PASSWORD}@127.0.0.1:5432/postgres'

- postgres_15_database_pg_statio_user_tables
static_configs:
- targets:
db1: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/db1'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
db1: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/db1'
db1: 'pg://prometheus_sql_exporter:${PASSWORD}@127.0.0.1:5432/db1'

static_configs:
- targets:
db1: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/db1'
db2: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/db2'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
db2: 'pg://sql_exporter:${PASSWORD}@127.0.0.1:25432/db2'
db2: 'pg://prometheus_sql_exporter:${PASSWORD}@127.0.0.1:5432/db2'

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.

None yet

2 participants