Skip to content

Add postgres exporter docs to website - #3076

Draft
ArthurSens wants to merge 5 commits into
mainfrom
postgres-exporter-docs
Draft

Add postgres exporter docs to website#3076
ArthurSens wants to merge 5 commits into
mainfrom
postgres-exporter-docs

Conversation

@ArthurSens

Copy link
Copy Markdown
Member

100% AI generated for now, adds prometheus-community/postgres_exporter#1361 to the website :)

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>

@jan--f jan--f left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure why 7b2a05e is needed. I'm not against moving exporter related content under a new exporter focused section but including this here makes thing unnecessarily complex.

Including the postgres docs also seems broken. In the preview the link gives 404.

…porters section

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens
ArthurSens force-pushed the postgres-exporter-docs branch from 7b2a05e to bb0ee19 Compare August 26, 2026 22:15
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens

Copy link
Copy Markdown
Member Author

Addressed the feedback during the docs call :)

Including the postgres docs also seems broken. In the preview the link gives 404.

That is because there's no release tag of the Postgres exporter that includes the /docs folder. I've added the commit 752d5d0 that changes the logic from release tag to SHA just so things are visible, but I'd like to remove that commit before merging here

@ArthurSens

Copy link
Copy Markdown
Member Author

prometheus-community/postgres_exporter#1361 also needs to include FrontMatter headers

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.

2 participants