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

Conduit: prometheus metrics #1230

Merged
merged 2 commits into from
Sep 20, 2022

Conversation

Eric-Warehime
Copy link
Contributor

Summary

Add prometheus metrics to Conduit. These should be fully backwards compatible w/ Indexer's metrics so that we can compare them 1 to 1 in most ways.

Test Plan

Generating metrics using block-generator.

@Eric-Warehime Eric-Warehime added Not-Yet-Enabled Feature is not yet enabled at this time Skip-Release-Notes Reserved for PRs which do not need to be included in Release Notes labels Sep 19, 2022
@codecov
Copy link

codecov bot commented Sep 19, 2022

Codecov Report

Merging #1230 (df10036) into conduit (7446007) will increase coverage by 0.02%.
The diff coverage is 70.00%.

@@             Coverage Diff             @@
##           conduit    #1230      +/-   ##
===========================================
+ Coverage    62.09%   62.11%   +0.02%     
===========================================
  Files           70       70              
  Lines         9387     9408      +21     
===========================================
+ Hits          5829     5844      +15     
- Misses        3059     3063       +4     
- Partials       499      501       +2     
Impacted Files Coverage Δ
conduit/pipeline.go 52.15% <64.70%> (+1.00%) ⬆️
importers/algod/algod_importer.go 78.57% <100.00%> (+0.79%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Eric-Warehime Eric-Warehime merged commit 7cf43e8 into algorand:conduit Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Not-Yet-Enabled Feature is not yet enabled at this time Skip-Release-Notes Reserved for PRs which do not need to be included in Release Notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants