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(ddm): Add custom_metrics_access organization flag #66953
Conversation
This PR has a migration; here is the generated SQL for --
-- Alter field flags on organization
--
-- (no-op) |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #66953 +/- ##
=======================================
Coverage 84.31% 84.32%
=======================================
Files 5309 5309
Lines 237399 237405 +6
Branches 41058 41060 +2
=======================================
+ Hits 200172 200185 +13
+ Misses 37009 37002 -7
Partials 218 218
|
e529721
to
8054d8c
Compare
@@ -431,6 +432,10 @@ def save(self): | |||
org.flags.codecov_access = data["codecovAccess"] | |||
if "require2FA" in data: | |||
org.flags.require_2fa = data["require2FA"] | |||
# This option is temporary since we will control custom metrics via a single feature flag in the future. This |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now, to enable custom metrics you have to activate:
"organizations:custom-metrics"
"organizations:ddm-ui"
"organizations:metrics-blocking"
"organizations:use-metrics-layer-in-alerts"
The role of this organization option is that it will be read by getsentry
and it will enable all of those flags automatically in case the option was set to true
. On self-hosted this org option will not be needed and rather the aforementioned flags will have to be activated manually.
This PR adds a new organization flag called
sentry:custom_metrics_access
which will be used internally bygetsentry
to enable certain feature flags needed by custom metrics.