-
Notifications
You must be signed in to change notification settings - Fork 87
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(normalization): Use exposed device-class-synthesis feature flag to gate device.class synthesis in light normalization #1974
Conversation
if config.device_class_synthesis_config { | ||
normalize_device_class(event); | ||
} |
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.
I assume this PR is related to #1970. This will only normalize device.class
when the feature flag is enabled for the project, so the functionality your other PR does won't run for organizations with the feature flag disabled. It will also skip other functionality, so no device.class
tags will exist on events. Is this what you want?
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.
Hi @iker-barriocanal , yup this is the effect we're looking for. We're starting to rollout device class to customers and we want to be able to turn off this functionality in case the classifications are wrong or something breaks.
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.
Please update the context of this PR to normalization
from mobile
, and address the other comments before merging. Thanks for contributing!
…te-device-class-synthesis-by-exposed-flag
Uses the exposed
organizations:device-class-synthesis
sentry feature flag to gate synthesis of thedevice.class
tag. This is a temporary flag for managing rollout to EA users and eventually to be removed for GA release.