feat(metrics): add total and per channel vods duration#1095
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds per-channel and total VOD duration metrics to the metrics system. Introduces two new gauge metrics (totalVodsDuration and channelVodDuration) to the Metrics struct, initializes them in NewService with Prometheus registration, and extends GatherMetrics to compute per-channel durations and aggregate totals. Changes
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Now that I think about it, I should probably add the storage distribution that we already have in the admin overview. |
Small PR to add the total vods duration to the metrics. I was wondering how days (months or years) of vods I had on my instance, so I tried to add it to the metrics export since it didn't seem very complex to add.
Visualization with Netdata of these exported values:
Per channel (pie)