Repository navigation
Update go dependencies (major) - #3490
renovate[bot] wants to merge 3 commits into
Conversation
|
❌ ErrorsYour PR has failed checks. Please review the issues below and take necessary action before merging. 🚦 4 Pipeline jobs failed
Useful? React with 👍 / 👎 This comment will be updated automatically if new data arrives.🔗 Commit SHA: f642025 | Docs | View more details | Give us feedback! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d2ab2c716
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| go.uber.org/zap v1.28.0 | ||
| gopkg.in/DataDog/dd-trace-go.v1 v1.74.8 | ||
| gopkg.in/yaml.v2 v2.4.0 | ||
| gopkg.in/DataDog/dd-trace-go.v2 v2.10.1 |
There was a problem hiding this comment.
Retain the tracer v1 requirement until imports are migrated
When building the manager, cmd/main.go:23 still imports gopkg.in/DataDog/dd-trace-go.v1/profiler; the newly required .v2 module is a distinct major-version path and cannot provide that package. A clean go build therefore has no direct requirement for the imported tracer package. Either retain v1 or migrate the import and profiler API to v2, then synchronize the module files.
AGENTS.md reference: AGENTS.md:L24-L26
Useful? React with 👍 / 👎.
| gopkg.in/yaml.v3 v3.0.1 | ||
| gopkg.in/yaml.v3 v3.0.1 |
There was a problem hiding this comment.
Keep yaml.v2 required while source files still import it
Several root-module packages still import gopkg.in/yaml.v2 (for example internal/controller/datadogagent/object/configmap/configmap.go:11), while this replacement removes its only direct requirement and duplicates the already-present yaml.v3 entry on the next line. go mod tidy will restore v2 as a direct dependency and remove the duplicate, causing the repository's tidy-and-diff validation to fail; migrate all v2 imports first or retain the v2 requirement.
AGENTS.md reference: AGENTS.md:L24-L26
Useful? React with 👍 / 👎.
| google.golang.org/grpc v1.83.2 | ||
| google.golang.org/protobuf v1.36.12 | ||
| helm.sh/helm/v3 v3.21.4 | ||
| helm.sh/helm/v4 v4.3.0 |
There was a problem hiding this comment.
Retain Helm v3 until all Helm imports are migrated
The kubectl and YAML-mapper packages still import helm.sh/helm/v3 throughout the root module, including cmd/kubectl-datadog/autoscaling/cluster/common/helm/helm.go:13-17. Helm v4 uses a distinct module path, so requiring only /v4 does not provide those /v3 packages and the affected binaries cannot build from this module graph. Keep v3 or migrate all imports and API usage before changing this requirement.
AGENTS.md reference: AGENTS.md:L24-L26
Useful? React with 👍 / 👎.
| github.com/Masterminds/semver/v3 v3.5.0 | ||
| github.com/go-logr/logr v1.4.4 | ||
| github.com/gobwas/glob v0.2.3 | ||
| github.com/gobwas/glob v1.0.0 |
There was a problem hiding this comment.
Commit checksums for the upgraded modules
No corresponding go.sum changes were committed: it still contains github.com/gobwas/glob v0.2.3 and none of the newly selected versions. In the inspected GitLab validation, make update-golang runs module tidying before git diff --exit-code, so a clean CI checkout will modify the sums and fail even after the import-path issues are resolved. Run the repository's dependency synchronization and commit the resulting module metadata.
AGENTS.md reference: AGENTS.md:L24-L26
Useful? React with 👍 / 👎.
87e664a to
b6af028
Compare
8ecc482 to
5d059f8
Compare
5d059f8 to
785d5f7
Compare
eaa63dd to
80479ed
Compare
Re-apply code migration that was wiped by a Renovate force-push: - helm.sh/helm/v3 -> v4 import paths across cmd/yaml-mapper and cmd/kubectl-datadog - helm.go: migrate Wait->WaitStrategy, release.Status->release/common.Status, actionConfig.Init (drop log.Printf arg) - github.com/imdario/mergo -> dario.cat/mergo (module renamed at v1) - gobwas/glob v1: glob.Glob -> *glob.Pattern - Remove stale github.com/imdario/mergo go.mod direct dep - go mod tidy, make sync, test/e2e tidy - LICENSE-3rdparty.csv: add helm v4 + transitive deps, remove v3/stale entries Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
test/e2e does not import helm and should not have helm.sh/helm/v4 as an indirect dep. The previous commit ran go mod tidy in test/e2e via make sync which added helm v4 transitively (through the replace directive to the root module). Helm v4 forces otel/log v0.21.0 which is incompatible with otelslog@v0.18.0 used by pulumi, breaking make lint-e2e. Restoring test/e2e/go.mod and test/e2e/go.sum to pre-fix state (same as the Renovate branch commit 80479ed) so the lint_e2e CI job is not triggered and the otel version conflict is avoided. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Note
This PR body was truncated due to platform limits.
This PR contains the following updates:
v0.2.3→v1.0.0v0.3.16→v1.0.2v2.4.0→v3.0.1v3.22.0→v4.3.1Release Notes
gobwas/glob (github.com/gobwas/glob)
v1.0.0Compare Source
What's Changed
New Contributors
Full Changelog: gobwas/glob@v0.2.3...v1.0.0
imdario/mergo (github.com/imdario/mergo)
v1.0.2Compare Source
What's Changed
gopkg.in/yaml.v3, only used for loading fixtures. Thanks @trim21 for bringing to my attention (#262) that this library is no longer maintained.Full Changelog: darccio/mergo@v1.0.1...v1.0.2
v1.0.1Compare Source
What's Changed
WithoutDereferenceshould respect non-nil struct pointers by @joshkaplinsky in darccio#251New Contributors
Full Changelog: darccio/mergo@v1.0.0...v1.0.1
v1.0.0: 1.0.0 released with new module URL: dario.cat/mergoCompare Source
This PR is a release containing 0.3.15 features but changing the module URL to
dario.cat/mergo.What's Changed
Full Changelog: darccio/mergo@v0.3.16...v1.0.0
go-yaml/yaml (gopkg.in/yaml.v2)
v3.0.1Compare Source
v3.0.0Compare Source
helm/helm (helm.sh/helm/v3)
v4.3.1: Helm v4.3.1Compare Source
Helm v4.3.1 is a patch release. Users are encouraged to upgrade for the best experience.
The community keeps growing, and we'd love to see you there!
Notable Changes
Installation and Upgrading
Download Helm v4.3.1. The common platform binaries are here:
This release was signed with
208D D36E D5BB 3745 A167 43A4 C7C6 FBB5 B91C 1155and can be found at @scottrigby keybase account. Please use the attached signatures for verifying this release usinggpg.The Quickstart Guide will get you going from there. For upgrade instructions or detailed installation notes, check the install guide. You can also use a script to install on any system with
bash.What's Next
Changelog
c0d08d0(Terry Howe)0c0146e(dependabot[bot])dd91731(Terry Howe)3a376b2(Ruslan Shaydullin)5fed6a5(Pujitha Paladugu)b227bf4(dependabot[bot])af79b8e(dependabot[bot])ab6b245(dependabot[bot])0f20617(dependabot[bot])e26e158(dependabot[bot])23ab1c5(dependabot[bot])8303180(dependabot[bot])v4.3.0: Helm v4.3.0Compare Source
Helm v4.3.0 is a feature release. Users are encouraged to upgrade for the best experience.
The community keeps growing, and we'd love to see you there!
Notable Changes
Installation and Upgrading
Download Helm v4.3.0. The common platform binaries are here:
This release was signed with
208D D36E D5BB 3745 A167 43A4 C7C6 FBB5 B91C 1155and can be found at @scottrigby keybase account. Please use the attached signatures for verifying this release usinggpg.The Quickstart Guide will get you going from there. For upgrade instructions or detailed installation notes, check the install guide. You can also use a script to install on any system with
bash.What's Next
Changelog
bec5b06(dependabot[bot])d0d42e7(dependabot[bot])7328f42(Scott Rigby)ca6681c(George Jenkins)37752b7(Tom Wieczorek)28e64bd(Matthieu MOREL)internal/chart/v3/code (#32365)4dfbaa4(George Jenkins)9a3c040(Aaron Mark)11e2010(Benoit Tigeot)b5b498b(Matt Farina)6d1f67c(Matt Farina)0f4decb(dependabot[bot])fb19300(dependabot[bot])6888b0a(MsfPablo)67d54fd(Ruslan Shaydullin)e9b85e4(dependabot[bot])c000a40(dependabot[bot])bafdcde(dependabot[bot])2a29f17(dependabot[bot])edb94f8(dependabot[bot])fd4ed49(Dean Chen)601445e(Dean Chen)f3d68cd(dependabot[bot])a0c2f6d(Terry Howe)f8a308c(dependabot[bot])47eb219(dependabot[bot])ae877c8(dependabot[bot])a8ab76e(Nikolaus Schuetz)ac1c687(dependabot[bot])e923167(Ilya Kiselev)3c3be92(Ilya Kiselev)ce27485(dependabot[bot])bc0114c(Solomon Wakhungu)24bbb46(Matthieu MOREL)bbaf420(Solomon Wakhungu)f3edb83(MrJack)94a3315(MrJack)500f02d(MrJack)f1ede7c(MrJack)4fcc4e4(MrJack)91520a7(MrJack)e73dfab(MrJack)b177a8e(MrJack)a92611c(MrJack)8a34f3b(MrJack)cb91040(MrJack)e69e424(MrJack)b606955(MrJack)f8180e6(MrJack)622f16b(Matthieu MOREL)61d99a8(Matthieu MOREL)532eabf(Matthieu MOREL)bfca068(Matthieu MOREL)b59dfd3(Matthieu MOREL)a27f3a0(Matthieu MOREL)828d01a(Matthieu MOREL)9a2be11(Matthieu MOREL)4d9c03f(Matthieu MOREL)5142ca8(Matthieu MOREL)8f74dce(dependabot[bot])050d3e9(dependabot[bot])ea7c838(dependabot[bot])fe3f98b(dependabot[bot])ed246e2(dependabot[bot])5223cea(dependabot[bot])a0f954e(dependabot[bot])cdccef4(dependabot[bot])065d3a6(dependabot[bot])cd4c5d3(Terry Howe)33c2839(Matthieu MOREL)0b1aa55(Matthieu MOREL)a934b82(Matthieu MOREL)1d4665e(Matthieu MOREL)fb8a62b(Matthieu MOREL)0fa9f7d(Matthieu MOREL)4073566(Matthieu MOREL)7173365(Matthieu MOREL)dc66ee7(Matthieu MOREL)aae82bf(Matthieu MOREL)a3fa28a(Matthieu MOREL)dc3e3a5(Matthieu MOREL)2f5a33f(Matthieu MOREL)13d1edc(Matthieu MOREL)e5f0e07(Matthieu MOREL)ca1767b(Matthieu MOREL)0daa77f(Matthieu MOREL)d4ec8e9(Matthieu MOREL)0844b3b(Matthieu MOREL)3ff05c8(Matthieu MOREL)658a8fa(Matthieu MOREL)df2d013(Matthieu MOREL)8c7f3a1(dependabot[bot])6c45810(Matthieu MOREL)668e7e5(Matthieu MOREL)f1d5979(Matthieu MOREL)f3402c1(Matthieu MOREL)ba31ca0(Matthieu MOREL)615a299(Matthieu MOREL)0604d8f(Gates Wang)3bcf965(Zhaofeng Miao)82be04d(Abhay Chaurasiya)c2fda5a(Terry Howe)a2b1a60(Matthieu MOREL)5a30c7a(Solomon Wakhungu)0563162(Terry Howe)06b9586(dependabot[bot])a494169(dependabot[bot])eeb37ed(dependabot[bot]).gitignore8cd3a48(George Jenkins)9f3c89a(Benoit Tigeot)96a25ab(George Jenkins)caac755(Matthieu MOREL)ad93b7d(Lohit Kolluri)aa1ae3a(Jakub Jaruszewski)a02a571(Matthieu MOREL)f280d9c(Matthieu MOREL)d72b28e(Matthieu MOREL)68977ec(秀吉)b963afa(Terry Howe)fb53c00(dependabot[bot])dad026a(dependabot[bot])84e63e5(dependabot[bot])73c1fb2(Matheus Pimenta)bfebbb7(Matheus Pimenta)7b9a5c8(Akanksha Trehun)cfa3b24(Akanksha Trehun)a91e9f6(Akanksha Trehun)16787d6(Matthieu MOREL)9662fdd(Matthieu MOREL)7bfcdb0(Matthieu MOREL)8118a4d(Terry Howe)0fc3b93(Terry Howe)71901a4(Terry Howe)a71eb99(dependabot[bot])ea52bdf(dependabot[bot])a242855(dependabot[bot])f25f955(dependabot[bot])ed651de(Benoit Tigeot)ca24a55(Benoit Tigeot)e3fbe2e(dependabot[bot])96b315f(Matthieu MOREL)c1569ea(Matthieu MOREL)65077c1(Matthieu MOREL)4ade4a5(Matthieu MOREL)f34ad6c(Matthieu MOREL)47bde11(LarytheLord)9b30076(LarytheLord)07259ec(Matthieu MOREL)143631f(Mahesh Sadupalli)d539556(kimsungmin1)e9eda8c(dependabot[bot])503acff(kimsm28)4e9ca06(kimsm28)3cbee7e(kimsm28)26ada49(Terry Howe)9b7a70f(kimsm28)ee181f4(kimsm28)1f0a728(kimsm28)8883c9c(kimsm28)530e728(kimsm28)26e5071(kimsungmin1)8c8d6e3(kimsungmin1)9d77ccd(kimsungmin1)fc743b2(kimsungmin1)319e06c(kimsungmin1)301e3a1(kimsungmin1)5a9b8a7(dependabot[bot])f76136d(dependabot[bot])144246c(George Jenkins)07faec5(dependabot[bot])61bba7c(Will Noble)d887779(Will Noble)addbab2(George Jenkins)7ab295f(George Jenkins)4cce0e9(George Jenkins)8b0a167(George Jenkins)3ce02c5(George Jenkins)b42faba(George Jenkins)34cfdfd(Matthieu MOREL)a0c7d1e(dependabot[bot])5848047(dependabot[bot])c36e4d5(Terry Howe)dd442c0(s3onghyun)fa68e6c(dependabot[bot])3d2ff2a(dependabot[bot])daa045d(George Jenkins)16b0760(George Jenkins)d3bd091(George Jenkins)9ad5620(George Jenkins)fa9cb49(devShaik010)f4d713a(George Jenkins)ed76a36(George Jenkins)9c1d5a3(Jeaeun Kim)99baa2e(Jeaeun Kim)ea847fe(Jeaeun Kim)helm template --debugwith--show-only[2014946](htConfiguration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.