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

feat(code/example): Add metrics to the example app's store #636

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

romac
Copy link
Member

@romac romac commented Dec 3, 2024

Closes: #635


PR author checklist

@romac romac marked this pull request as draft December 3, 2024 20:36
@romac romac added work in progress Work in progress good first issue Good for newcomers labels Dec 4, 2024
@romac romac changed the title chore(code/host): Add metrics to the block store feat(code/host): Add metrics to the block store Dec 19, 2024
@romac romac changed the title feat(code/host): Add metrics to the block store feat(code/example): Add metrics to the example app's store Jan 15, 2025
@romac romac force-pushed the romac/blockstore-metrics branch from 6c9e5dc to 9d3dcef Compare January 15, 2025 09:43
OakenKnight and others added 2 commits January 17, 2025 15:23
* added metrics to example store

* optimized storing metrics in example app

* added time observing metrics to example app

* moving SharedRegistry import below

* serving metrics in app_channel

* renamed channel_app prefix from blockstore to app_channel

* Handle errors in metrics server

* Improve metrics collection in store

* Formatting

---------

Co-authored-by: Romain Ruetschi <[email protected]>
@romac romac marked this pull request as ready for review January 17, 2025 14:46
@romac romac removed good first issue Good for newcomers work in progress Work in progress labels Jan 17, 2025
@romac romac merged commit 7e12159 into main Jan 17, 2025
12 checks passed
@romac romac deleted the romac/blockstore-metrics branch January 17, 2025 15:23
Copy link

codecov bot commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.82%. Comparing base (775b934) to head (7912b1e).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #636      +/-   ##
==========================================
- Coverage   76.87%   76.82%   -0.05%     
==========================================
  Files         170      170              
  Lines       14389    14399      +10     
==========================================
  Hits        11061    11061              
- Misses       3328     3338      +10     
Flag Coverage Δ
integration 76.49% <ø> (-0.05%) ⬇️
mbt 21.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

code: Add metrics to example app's store
2 participants