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(dynamic-sampling): Add isSendingSource
to dynamic-sampling SDK versions endpoint [TET-271]
#37295
feat(dynamic-sampling): Add isSendingSource
to dynamic-sampling SDK versions endpoint [TET-271]
#37295
Conversation
…SamplingSDKVersionsEndpoint` Add new attribute to dymanic-sampling SDK versions endpoint resolves [TET-271]
isSendingSource
to OrganizationDynamicSamplingSDKVersionsEndpoint
isSendingSource
to dynamic-sampling SDK versions endpoint [TET-271]
transaction_name="transaction_source", | ||
# Only available in transactions, pretend like its a tag so we dont error and just null instead | ||
discover_name="tags[transaction_source]", | ||
discover_name="transaction_source", |
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.
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.
AFAIK transaction_source isn't in the discover dataset, and only in the transactions dataset?
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.
fixed.
avg_equation = 'count_if(trace.client_sample_rate, notEquals, "") / count()' | ||
sample_rate_count_if = 'count_if(trace.client_sample_rate, notEquals, "")' | ||
avg_sample_rate_equation = f"{sample_rate_count_if} / count()" | ||
transaction_source_count_if = 'count_if(transaction.source, Equals, "unknown")' |
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.
here i still need clarification, what's our target here.
123ceb7
to
0a6ada5
Compare
src/sentry/api/endpoints/organization_dynamic_sampling_sdk_versions.py
Outdated
Show resolved
Hide resolved
src/sentry/api/endpoints/organization_dynamic_sampling_sdk_versions.py
Outdated
Show resolved
Hide resolved
…sions.py Co-authored-by: Jan Michael Auer <mail@jauer.org>
…sions.py Co-authored-by: Jan Michael Auer <mail@jauer.org>
e483790
to
ce259fd
Compare
"project": [self.project.id], | ||
} | ||
response = self.do_request(query) | ||
assert response.status_code == 200, response.content |
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.
we just need to wait for new snuba version.
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.
this test rely on this snuba commit getsentry/snuba@9e7b0ec
Add new attribute to dynamic-sampling SDK versions
endpoint
OrganizationDynamicSamplingSDKVersionsEndpoint
.related snuba PR getsentry/snuba#3020
resolves TET-271