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

[BUG] v1.44.0 seems to pull in a broken pkg #1607

Closed
Tethrinaa opened this issue Dec 6, 2022 · 7 comments · Fixed by #1613 or #1620
Closed

[BUG] v1.44.0 seems to pull in a broken pkg #1607

Tethrinaa opened this issue Dec 6, 2022 · 7 comments · Fixed by #1613 or #1620
Assignees
Labels
bug unintended behavior that has to be fixed

Comments

@Tethrinaa
Copy link

Describe what happened:
Using go 1.19, running go get -u upgraded our codebase from v1.43.1 => v1.44.0, this seems to have pulled in:
go: added github.com/DataDog/datadog-agent/pkg/remoteconfig/state v0.40.1

Which causes the following build error:

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

Manually reverting to v1.43.1 resolved the issue. Some kind of linking issue with 1.44.0?

@Tethrinaa Tethrinaa added the bug unintended behavior that has to be fixed label Dec 6, 2022
@qgxpagamentos
Copy link

qgxpagamentos commented Dec 6, 2022

we are having the same problem.
how did you downgrade? defined in go.mod some version?
@Tethrinaa

@Tethrinaa
Copy link
Author

Yes, I manually changed the go mod line to:
gopkg.in/DataDog/dd-trace-go.v1 v1.43.1 // indirect
and then run go mod tidy

@ajgajg1134 ajgajg1134 self-assigned this Dec 6, 2022
@ajgajg1134
Copy link
Contributor

👋 Hello! Thanks for bringing this up. It looks like the latest version of that remoteconfig package isn't quite compatible with v1.44.0 of dd-trace-go. I will take a look tomorrow at resolving the remoteconfig package issue inside the datadog-agent (it seems like it might just need a new tag)

If you'd like to upgrade dd-trace-go now and not wait for that fix you can use go get -u gopkg.in/DataDog/dd-trace-go.v1@v1.44.0 which will pull a working version of remoteconfig/state and not upgrade it to v0.40.1

@Julio-Guerra
Copy link
Contributor

Hey everyone,
Sorry again about this issue.

dd-trace-go v1.44.1 is out and should fix your issue.
Could you please give it a try by updating dd-trace-go the way you used to (I guess either go get -u all or go get -u) and let us know?

@sindrig
Copy link

sindrig commented Dec 13, 2022

I'm getting the same errors after automatic dependency updates.

Your nightly build agrees with me: https://github.com/DataDog/dd-trace-go/actions/runs/3679384649/jobs/6223776294 :(

@Julio-Guerra Julio-Guerra reopened this Dec 13, 2022
@Julio-Guerra
Copy link
Contributor

I'm getting the same errors after automatic dependency updates.

Thanks for bringing this to our attention. Our fix got lost in a git tag that followed it :-/
We are looking into properly tagging it again. The fix will be part of the v1.45.0 release happening later this week.

@Julio-Guerra
Copy link
Contributor

v1.45.0-rc.2 is out and now includes this fix. We should release it next Thursday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unintended behavior that has to be fixed
Projects
None yet
5 participants