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

Updated metrics file with new polygon related panels #13309

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

eastorski
Copy link
Member

No description provided.

@eastorski eastorski requested a review from dvovk January 3, 2025 09:55
@dvovk dvovk self-requested a review January 3, 2025 10:23
Copy link
Member

@dvovk dvovk left a comment

Choose a reason for hiding this comment

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

"__inputs" - should be here
check the checkbox:
Screenshot 2025-01-03 at 10 28 18

@eastorski eastorski force-pushed the eastorski/polygon-metrics branch from de1f9f0 to 161460b Compare January 3, 2025 10:34
@eastorski eastorski requested a review from dvovk January 3, 2025 10:35
@eastorski eastorski force-pushed the eastorski/polygon-metrics branch from 161460b to f8dfdd5 Compare January 3, 2025 11:28
@eastorski eastorski force-pushed the eastorski/polygon-metrics branch from f8dfdd5 to ff749ee Compare January 3, 2025 11:30
@dvovk dvovk merged commit e727649 into main Jan 3, 2025
13 checks passed
@dvovk dvovk deleted the eastorski/polygon-metrics branch January 3, 2025 12:15
taratorio added a commit that referenced this pull request Jan 6, 2025
removes outdated charts we added as part of
#12927 because:
1. these were changed and superseded by new versions in erigon_internals
as part of #13309
2. no need to duplicate these both in
`prometheus/dashboards/erigon.json` and
`prometheus/dashboards/erigon_internals.json` - only
`erigon_internals.json` is enough
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