Skip to content
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

exporter/datadogexporter: bump datadog-agent dependencies #14400

Merged
merged 2 commits into from
Sep 22, 2022

Conversation

gbbr
Copy link
Member

@gbbr gbbr commented Sep 22, 2022

This change bumps the datadog-agent dependencies to contain the latest changes re. removing deprecated pdata methods.

@gbbr gbbr requested a review from a team September 22, 2022 12:46
@gbbr gbbr requested a review from mx-psi as a code owner September 22, 2022 12:46
@bogdandrutu bogdandrutu added the dependencies Pull requests that update a dependency file label Sep 22, 2022
@bogdandrutu
Copy link
Member

Can you rebase please?

@gbbr
Copy link
Member Author

gbbr commented Sep 22, 2022

Done. Thanks for reviewing.

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Looks like some more changes are needed here

../../../../go/pkg/mod/github.com/!data!dog/datadog-agent/pkg/remoteconfig/[email protected]/tuf.go:34:20: rootClient.InitLocal undefined (type *client.Client has no field or method InitLocal)

@gbbr
Copy link
Member Author

gbbr commented Sep 22, 2022

Thanks Alex. I reverted that change since it wasn't critical. We'll do it separately. Should be ok now.

@codeboten codeboten merged commit 1d61501 into open-telemetry:main Sep 22, 2022
@gbbr gbbr deleted the gbbr/bump-exporter branch September 27, 2022 18:05
@gbbr gbbr restored the gbbr/bump-exporter branch September 30, 2022 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants