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

PMM-7806: add compatibility with percona pg exporter #68

Merged
merged 19 commits into from
Sep 6, 2022

Conversation

ritbl
Copy link

@ritbl ritbl commented May 22, 2022

@@ -0,0 +1,1049 @@
# HELP go_gc_duration_seconds A summary of the pause duration of garbage collection cycles.
Copy link
Author

@ritbl ritbl May 22, 2022

Choose a reason for hiding this comment

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

Baseline, output from Percona Exporter running in pmm devcontainer (2.28.0)

referenceMetrics := toMap(t, referenceMetrics)

//remove matches
for m, _ := range currentMetrics {

Choose a reason for hiding this comment

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

🚫 [golangci-lint] reported by reviewdog 🐶
S1005: unnecessary assignment to the blank identifier (gosimple)

@ritbl ritbl temporarily deployed to CI June 3, 2022 12:04 Inactive
@ritbl ritbl temporarily deployed to CI June 3, 2022 12:04 Inactive
@ritbl ritbl temporarily deployed to CI June 3, 2022 12:04 Inactive
@ritbl ritbl temporarily deployed to CI June 3, 2022 12:04 Inactive
@ritbl ritbl temporarily deployed to CI June 3, 2022 12:04 Inactive
@ritbl ritbl temporarily deployed to CI June 14, 2022 09:34 Inactive
@ritbl ritbl temporarily deployed to CI June 14, 2022 09:34 Inactive
@ritbl ritbl temporarily deployed to CI June 14, 2022 09:34 Inactive
@ritbl ritbl temporarily deployed to CI June 14, 2022 09:34 Inactive
@ritbl ritbl temporarily deployed to CI June 14, 2022 09:34 Inactive
@@ -0,0 +1,143 @@
package main
Copy link
Member

Choose a reason for hiding this comment

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

Was this file created to make it easier to support upstream changes?

Choose a reason for hiding this comment

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

yep, to separate our code from upstream

@tshcherban tshcherban merged commit db4c6e8 into main-upstream Sep 6, 2022
@tshcherban tshcherban deleted the PMM-7806-upgrade-pg-exporter branch September 6, 2022 09:37
ritbl added a commit to heniek/postgres_exporter that referenced this pull request Mar 19, 2023
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.

6 participants