fix(API Usage): Allow self-hosted API usage alerts#6975
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
5455d1b to
5b1d009
Compare
9801c2a to
fc9e700
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6975 +/- ##
==========================================
- Coverage 98.32% 98.32% -0.01%
==========================================
Files 1335 1335
Lines 49876 49850 -26
==========================================
- Hits 49039 49013 -26
Misses 837 837 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit in Settings → Usage.
Once credits are available, reopen this pull request to trigger a review.
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
khvn26
left a comment
There was a problem hiding this comment.
Looks good in principle. I'd love to see more docs on the feature flag usage here (rationale, rollout plan).
I've updated the issue description with a [hopefully] clear rollout plan. |
I have added information todocs/if required so people know about the feature.Changes
Contributes to https://github.com/Flagsmith/infrastructure/issues/334
Deprecate
get_current_api_usage(Influx) in favour ofget_total_events_count(Influx/Postgres).get_current_api_usage_deprecatedThis is part of a plan to enable usage alert emails for self-hosted users, by [experimentally] reading usage data from local Postgres instead of InfluxDB — our usage backend in production.
How did you test this code?
Tests incoming as part of a private cloud endeavour.