Update all github.com/datadog packages #31398
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
liustanley
requested review from
a team,
jpkrohling,
MovieStoreGuy,
mx-psi,
songy23 and
dmitryax
as code owners
github-actions
bot
added
cmd/configschema
github-actions
bot
requested review from
boostchicken,
dineshg13,
gouthamve and
mackjmr
liustanley
commented
Feb 23, 2024
dineshg13
reviewed
Feb 23, 2024
jpkrohling
approved these changes
Feb 26, 2024
There was a problem hiding this comment.
Reviewing on behalf of datadog-receiver. I'll defer to @mx-psi for the other ones.
This PR doesn't have any user facing changes, so no changelog is needed. |
1 task
liustanley
changed the title
dineshg13
approved these changes
Feb 27, 2024
dineshg13
reviewed
Feb 27, 2024
songy23
approved these changes
Feb 27, 2024
|
Please rebase |
1 task
songy23
added
the
ready to merge
songy23
removed
the
ready to merge
mx-psi
approved these changes
Feb 29, 2024
liustanley
changed the title
songy23
approved these changes
Feb 29, 2024
codeboten
approved these changes
Feb 29, 2024
XinRanZhAWS
pushed a commit
to XinRanZhAWS/opentelemetry-collector-contrib
that referenced
this pull request
Mar 13, 2024
Updates all datadog dependencies to the psuedo version from the [otel/backport](https://github.com/DataDog/datadog-agent/tree/otel/backport) branch and addresses the following breaking changes: - DataDog/datadog-agent#22729 - DataDog/datadog-agent#22809 Functional changes: - Initialize `OTLPReceiver.AttributesTranslator` in traces pipeline, and set in trace agent configs - Metrics client is no longer initalized in `createTracesToTracesConnector` because there is no initialization of the trace agent in the trace to trace connector - `InitializeMetricClient` can be called multiple times now that the singleton is gone - `metrics_client.go` implements `Distribution` by calling `Histogram` for the same implementation - `metrics_client.go` adds placeholders for statsd methods that are not applicable to our collector components
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Updates all datadog dependencies to the psuedo version from the otel/backport branch and addresses the following breaking changes:
Functional changes:
OTLPReceiver.AttributesTranslatorin traces pipeline, and set in trace agent configscreateTracesToTracesConnectorbecause there is no initialization of the trace agent in the trace to trace connectorInitializeMetricClientcan be called multiple times now that the singleton is gonemetrics_client.goimplementsDistributionby callingHistogramfor the same implementationmetrics_client.goadds placeholders for statsd methods that are not applicable to our collector componentsLink to tracking Issue:
Testing:
Documentation:
This PR doesn't have any user facing changes, so no changelog is needed.