Skip to content

[chore] Upgrade datadog-agent and mapping go versions and fix breaking changes #24674

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Aug 7, 2023
Merged

[chore] Upgrade datadog-agent and mapping go versions and fix breaking changes #24674

merged 6 commits into from
Aug 7, 2023

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Jul 29, 2023 •
edited
Loading

Description:
Roll forward #24552 with stable versions of dependencies.

Upgrade dependency versions of github.com/DataDog/opentelemetry-mapping-go and github.com/DataDog/datadog-agent and fix breaking changes in protobuf (context: DataDog/datadog-agent#10650, DataDog/opentelemetry-mapping-go#126).

Link to tracking Issue:

Testing:

Documentation:

@songy23 songy23 added dependencies Pull requests that update a dependency file Skip Changelog PRs that do not require a CHANGELOG.md entry labels Jul 29, 2023
mx-psi
mx-psi reviewed Jul 31, 2023
View reviewed changes
@songy23 songy23 force-pushed the bump-ddog-version branch from f5aa518 to 80e1770 Compare July 31, 2023 16:33
@github-actions github-actions bot requested a review from mackjmr August 2, 2023 16:35
@songy23 songy23 mentioned this pull request Aug 3, 2023
Merged
@songy23 songy23 force-pushed the bump-ddog-version branch from c711098 to dd7ba5f Compare August 4, 2023 19:25
@songy23 songy23 force-pushed the bump-ddog-version branch from dd7ba5f to 9911a97 Compare August 4, 2023 21:08
@songy23 songy23 removed the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 4, 2023
@songy23 songy23 marked this pull request as ready for review August 4, 2023 21:10
@songy23 songy23 requested review from a team and mx-psi August 4, 2023 21:10
@mx-psi
Copy link
Member

mx-psi commented Aug 7, 2023

@dmitryax do you want to review before merging?

@mx-psi
Copy link
Member

mx-psi commented Aug 7, 2023

Needs a rebase after merging the internal/datadog module PR

dmitryax
dmitryax approved these changes Aug 7, 2023
View reviewed changes
Copy link
Member

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@dmitryax dmitryax merged commit ff60858 into open-telemetry:main Aug 7, 2023
@github-actions github-actions bot added this to the next release milestone Aug 7, 2023
@songy23 songy23 deleted the bump-ddog-version branch August 7, 2023 16:34
@dmitryax dmitryax mentioned this pull request Aug 9, 2023
Merged
@daianmartinho daianmartinho mentioned this pull request Nov 29, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mx-psi mx-psi mx-psi approved these changes

@dmitryax dmitryax dmitryax approved these changes

@boostchicken boostchicken Awaiting requested review from boostchicken

@dineshg13 dineshg13 Awaiting requested review from dineshg13

@gbbr gbbr Awaiting requested review from gbbr

@gouthamve gouthamve Awaiting requested review from gouthamve

@jpkrohling jpkrohling Awaiting requested review from jpkrohling

@liustanley liustanley Awaiting requested review from liustanley

@MovieStoreGuy MovieStoreGuy Awaiting requested review from MovieStoreGuy MovieStoreGuy is a code owner

@pmcollins pmcollins Awaiting requested review from pmcollins

@mackjmr mackjmr Awaiting requested review from mackjmr

Assignees

@mx-psi mx-psi

Labels
cmd/configschema configschema command cmd/otelcontribcol otelcontribcol command dependencies Pull requests that update a dependency file exporter/datadog Datadog components internal/datadog processor/datadog receiver/datadog testbed
Projects
None yet
Milestone
v0.83.0
Development

Successfully merging this pull request may close these issues.
3 participants
@songy23 @mx-psi @dmitryax