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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable postgres_exporter by default #516

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

nhudson
Copy link
Contributor

@nhudson nhudson commented Jul 29, 2022

Enable postgres_exporter by default

Description

We want to enable the postgres_exporter provided from timescaledb-single Helm chart by default. This will allow us to collect metrics from the DB by default.

  • Add in postgres-exporter mixin alerts and Grafana dashboard
  • Fix and merge how we build and generate mixin dashboards and alerts

Type of change

What type of changes does your code introduce to tobs? Put an x in the box that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

@nhudson nhudson self-assigned this Jul 29, 2022
@nhudson nhudson force-pushed the add_postgres_exporter branch from b59dbe6 to 75e3735 Compare August 2, 2022 14:43
@nhudson
Copy link
Contributor Author

nhudson commented Aug 2, 2022

The mixin Dashboard seems to have some issues specifically around thhe datasource, as it's not really labeled in a consistent way. See prometheus-community/postgres_exporter#647 for a bit more context.

@nhudson nhudson marked this pull request as ready for review August 2, 2022 20:28
@nhudson nhudson requested a review from a team as a code owner August 2, 2022 20:28
@nhudson nhudson linked an issue Aug 2, 2022 that may be closed by this pull request
@nhudson nhudson requested review from paulfantom and a team August 3, 2022 15:53
@nhudson nhudson force-pushed the add_postgres_exporter branch 2 times, most recently from 64acdd5 to 4e50499 Compare August 3, 2022 17:26
@nhudson nhudson enabled auto-merge August 3, 2022 17:28
@nhudson nhudson force-pushed the add_postgres_exporter branch from 4e50499 to 9047c05 Compare August 4, 2022 14:07
@nhudson nhudson force-pushed the add_postgres_exporter branch from 9047c05 to 3960721 Compare August 8, 2022 15:05
  - Add in postgres-exporter mixin alerts and Grafana dashboard
  - Fix and merge how we build and generate mixin dashboards and alerts
@nhudson nhudson force-pushed the add_postgres_exporter branch from 3b02156 to 9692966 Compare August 8, 2022 15:32
@nhudson nhudson merged commit 9fce092 into timescale:main Aug 8, 2022
@nhudson nhudson deleted the add_postgres_exporter branch August 8, 2022 15:38
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.

Enable postgres_exporter by default
2 participants