Statistics Tracking - #75
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #75 +/- ##
==========================================
- Coverage 87.86% 87.35% -0.51%
==========================================
Files 8 8
Lines 2430 2506 +76
Branches 2430 2506 +76
==========================================
+ Hits 2135 2189 +54
- Misses 261 282 +21
- Partials 34 35 +1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Test Results46 tests 46 ✅ 2s ⏱️ Results for commit 3e5934c. ♻️ This comment has been updated with latest results. |
b0b2e08 to
876fb25
Compare
969ee51 to
93638aa
Compare
e1614e8 to
93d5271
Compare
|
Why not export metrics in OTel or Prometheus format so we can integrate with dashboards? |
|
I don't know anything about either of those; could you point me to what I should read? |
Both are systems for collecting/processing metrics from software. Prometheus: https://prometheus.io/docs/introduction/overview/ OpenTelemetry (intro): https://opentelemetry.io/docs/what-is-opentelemetry/ Originally the plan was to add Prometheus metrics to collect standard performance/behavior metrics about the process, but it sounded like you wanted to collect other statistics (more like user behavior) so I left that up to you. We did also discuss Prometheus a little in Discord: https://discord.com/channels/772576325897945119/1319140464812753009/1373470039478370385 Basically it boils down to a standard format for exporting metrics so we can use existing software to collect/process/display the data instead of having to re-implement all of that. There are also application-side libraries for Prometheus (and I assume OTel too) that make it easy to calculate and export more complex metrics like histograms of response latencies. |
|
Ok sick, I will look into those, though I'm highkey locked in for spill so I may look at that over the winter |
93d5271 to
b760b11
Compare
Gitignore the webring.log Implement clearing the statistics data daily
b760b11 to
3e5934c
Compare
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
Closes #50
MVP
Important but maybe a future PR