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

fix: API usage alerting in production #3507

Merged
merged 28 commits into from
Mar 28, 2024
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
f0c6406
WIP: use flagsmith to filter organisations to include in usage alerting
matthewelwell Feb 27, 2024
781c474
Move get client to outside of loop
zachaysan Feb 28, 2024
5989a60
Implement local evaluation for flagsmith client
zachaysan Feb 28, 2024
888022c
Fix influxdb query call
zachaysan Feb 28, 2024
3715eb3
Merge branch 'main' into fix/api-usage-alerting-in-production
zachaysan Feb 28, 2024
a2b75c5
Trigger build
zachaysan Feb 28, 2024
a5dc209
Trigger build
zachaysan Feb 28, 2024
c8b39f8
Update tests to pass CI
zachaysan Mar 1, 2024
761c372
Add comment with PR
zachaysan Mar 13, 2024
3947f6a
Use a local for easier debugging
zachaysan Mar 13, 2024
f52b9b3
Create test with feature flag being toggled off and add fine grain cl…
zachaysan Mar 13, 2024
c882d34
Merge branch 'main' into fix/api-usage-alerting-in-production
zachaysan Mar 13, 2024
833a17a
Remove temporary code
zachaysan Mar 14, 2024
8157896
Update dependencies
zachaysan Mar 15, 2024
268c46d
Trigger build
zachaysan Mar 15, 2024
6db904a
Trigger build
zachaysan Mar 15, 2024
32c7b06
Reduce whitespace to test CI
zachaysan Mar 15, 2024
0210383
Fix conflicts and merge branch 'main' into fix/api-usage-alerting-in-…
zachaysan Mar 15, 2024
352f299
Update due to library versioning
zachaysan Mar 15, 2024
031449f
Add new data since upgrade
zachaysan Mar 15, 2024
a69929c
Add composite key
zachaysan Mar 15, 2024
77b3e1d
Freeze time
zachaysan Mar 15, 2024
091b010
Set environment id to string for pydantic
zachaysan Mar 15, 2024
585710f
Add IdentityFeaturesList to test
zachaysan Mar 18, 2024
eedcc37
Avoid double iteration
zachaysan Mar 18, 2024
1f07ee1
Fix conflicts, add fixture, and merge branch 'main' into fix/api-usag…
zachaysan Mar 18, 2024
136bdd1
Merge branch 'fix/api-usage-alerting-in-production' of github.com:Fla…
zachaysan Mar 18, 2024
2d3b46c
Fix conflicts and merge branch 'main' into fix/api-usage-alerting-in-…
zachaysan Mar 27, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion api/edge_api/identities/edge_identity_service.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,7 @@ def get_edge_identity_overrides(
environment_id=environment_id, feature_id=feature_id
)
)
return [IdentityOverrideV2.parse_obj(item) for item in override_items]
for item in override_items:
item["environment_id"] = str(item["environment_id"])

return [IdentityOverrideV2.model_validate(item) for item in override_items]