-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
pkg/trace/api/otlp: support APM peer.service in OTLP #16490
Conversation
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.
Bloop Bleep... Dogbot HereRegression Detector ResultsRun ID: 3c2d7410-68d2-44dd-aa2c-429638ec4d87 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. No interesting changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%. Fine details of change detection per experiment.
|
c1500c3
to
7d4dc82
Compare
Finally was able to build a local agent (thanks to @dineshg13 !). Tested in staging that non top level OTLP spans also get trace metrics:
|
36b6fbb
to
ce2bdb3
Compare
What does this PR do?
Implement OTLP support for APM
peer.service
in trace agent:peer.service
defaults in cases wherepeer.service
is missing.peer.service
provenance tag_dd.peer.service.source={source_attribute}
service={peer.service}
ifpeer.service
is non-emptyspan.kind
is set correctly as the agent relies on this attribute to decide whether or not to calculate stats.Motivation
To be compliant with the APM peer.service RFC and implementation (#16103).
Additional Notes
Possible Drawbacks / Trade-offs
This will be a breaking change to users who depend on overriding
Service
with span attributepeer.service
.Describe how to test/QA your changes
peer.service
to Agent, then verify the DD span meta tag_dd.peer.service.source
is set and DDService
isn't overridden withpeer.service
.net.peer.name
or any other default key to Agent, then verify the DD span meta tagpeer.service
and_dd.peer.service.source
are set.span.kind
is set.Reviewer's Checklist
Triage
milestone is set.major_change
label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.changelog/no-changelog
label has been applied.qa/skip-qa
label is not applied.team/..
label has been applied, indicating the team(s) that should QA this change.need-change/operator
andneed-change/helm
labels have been applied.k8s/<min-version>
label, indicating the lowest Kubernetes version compatible with this feature.