Update containers postgresql_details.json dashboard to use pgMonitor …#470
Merged
keithf4 merged 1 commit intoCrunchyData:developmentfrom May 28, 2025
Merged
Conversation
keithf4
previously approved these changes
May 23, 2025
Contributor
|
@dsessler7 are there any changes that need to be made to the monitoring_installers repo for the v5.2.1 update? |
hunleyd
previously approved these changes
May 23, 2025
Contributor
Author
…v5.2.1 metrics. Update other containers dashboards to use ccp metrics and maintain compatibility with postgres_exporter metrics. Add pgBouncer dashboard. Ignore pgBouncer in non-pgBouncer dashboards.
3609d96 to
1bf5620
Compare
Contributor
Author
|
Nevermind, I found the changelogs README and added one. Hope that suffices... However it looks like I need another approval. |
hunleyd
approved these changes
May 27, 2025
keithf4
approved these changes
May 28, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…v5.2.1 metrics.
Update other containers dashboards to use ccp metrics and maintain compatibility with postgres_exporter metrics. Add pgBouncer dashboard.
Ignore pgBouncer in non-pgBouncer dashboards.
Description
Please fill this template out fully; failure to do so can result in rejection of your PR.
Please describe the changes made by your PR. This description should be at a high-level but detailed enough that a reviewer understands the scope of the fix or enhancement and can easily judge the PRs validity at addressing the stated issue/feature. Please fully describe any new or changed feature and whether said change is user-facing or not:
The OpenTelemetry metrics feature in CPK is designed to use the latest available ccp metrics in pgMonitor. This PR updates the containers dashboards to use the metrics while also maintaining compatibility with postgres_exporter metrics. This PR also adds a pgBouncer dedicated dashboard and ignores pgBouncer pods in other non-pgBouncer specific dashboards.
Please indicate what kind of change your PR includes (multiple selections are acceptable):
PRs should be against existing issues, so please list each issue using a separate 'closes' line:
closes #469
If this PR depends on another PR or resolution of another issue, please indicate that here using a separate 'depends' line for each dependency.
N/A
If you have an external dependency (packages, portal updates, etc), add the 'BLOCKED' tag to your PR.
Testing
None of the testing listed below is optional.
Code testing
Have you tested your changes against:
If your code touches sql_exporter, have you:
If your code touches node_exporter, have you:
If your code touches Prometheus, have you:
promtool check configpromtool check rulesIf your code touches Grafana, have you:
Checklist: