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(profiling): pass internal (profile-scoped) dsn for function metrics ingestion to vroom #66300
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #66300 +/- ##
==========================================
- Coverage 84.29% 84.29% -0.01%
==========================================
Files 5309 5309
Lines 237222 237235 +13
Branches 41038 41041 +3
==========================================
+ Hits 199972 199982 +10
- Misses 37031 37034 +3
Partials 219 219
|
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 cannot assess whether Vroom can handle the extra "options"
key in the profile, but LGTM!
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.
worth adding tests for this?
…ofiling-metrics-dsn
if options.get( | ||
"profiling.generic_metrics.functions_ingestion.enabled" | ||
) and project.organization_id in options.get( | ||
"profiling.generic_metrics.functions_ingestion.allowed_org_ids" | ||
): |
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.
As discussed, we should move this to a feature flag using the options backed rollout handler for a more incremental rollout.
But for initial testing, this is fine 👍
…ics ingestion to vroom (#66300)
No description provided.