Skip to content

Conversation

@mshahzeb
Copy link

Add variables for grouping - Postgres mixin

@v-zhuravlev v-zhuravlev self-requested a review August 20, 2024 04:34
Copy link

@shanavazh shanavazh left a comment

Choose a reason for hiding this comment

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

LGTM

},
expr: |||
avg by (datname) (
avg by (%(agg)s) (

Choose a reason for hiding this comment

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

we need datname here, like other rules below

@mshahzeb mshahzeb merged commit 1b9cff9 into master Aug 21, 2024
cristiangreco pushed a commit to prometheus-community/postgres_exporter that referenced this pull request Jul 17, 2025
cristiangreco pushed a commit to prometheus-community/postgres_exporter that referenced this pull request Jul 21, 2025
SuperQ pushed a commit to prometheus-community/postgres_exporter that referenced this pull request Jul 25, 2025
* Update mixin to latest changes from grafana/postgres_exporter

Porting a bunch of PRs accumulated over time in grafana/postgres_exporter:

- grafana#11 (@v-zhuravlev)
- grafana#12 (@gaantunes)
- grafana#13 (@gaantunes)
- grafana#14 (@gaantunes)
- grafana#15 (@gaantunes)
- grafana#16 (@gaantunes)
- grafana#17 (@gaantunes)
- grafana#20 (@gaantunes)
- grafana#21 (@mshahzeb)
- grafana#22 (@mshahzeb)

Signed-off-by: Cristian Greco <[email protected]>

* rename dashboard to old name

Signed-off-by: Cristian Greco <[email protected]>

* remove custom selector

Signed-off-by: Cristian Greco <[email protected]>

---------

Signed-off-by: Cristian Greco <[email protected]>
Co-authored-by: Vitaly <[email protected]>
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.

3 participants