Update dependencies - #24692
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
Validation ReportAll 21 validations passed. Show details
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 98a99553bc
ℹ️ 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".
| [project.optional-dependencies] | ||
| deps = [ | ||
| "redis==7.3.0", | ||
| "redis==8.0.1", |
There was a problem hiding this comment.
Keep redis-py on the RESP2-compatible major
For the Redis 5.0 environment explicitly retained in redisdb/hatch.toml, redis-py 8 enables the RESP3/HELLO connection behavior while Redis 5 predates that command. _get_conn constructs redis.Redis without a protocol override, so the supported Redis 5 check fails during connection setup before collecting metrics. Commit 283cdf26 already reverted this same redis 8 bump because it broke redisdb; retain 7.3 or add compatible protocol handling before upgrading.
Useful? React with 👍 / 👎.
| cm-client==45.0.4 | ||
| confluent-kafka==2.13.2 | ||
| cryptography==48.0.1 | ||
| cryptography==49.0.0 |
There was a problem hiding this comment.
Do not repin cryptography 49 for Intel macOS
In the resolve-build-deps.yaml macOS x86_64 target, cryptography 49 has no compatible Intel/universal wheel, so resolution falls back to an sdist build that fails after the workflow removes the OpenSSL/pkg-config build prerequisites. The repository explicitly pinned <49 for this target in commit 560b68ff, and commit 283cdf26 reverted this exact Agent dependency bump after it broke TLS and the macOS x86_64 build; this pin reintroduces that failure.
Useful? React with 👍 / 👎.
| in-toto==2.0.0 | ||
| jellyfish==1.2.1 | ||
| kubernetes==35.0.0 | ||
| kubernetes==36.0.3 |
There was a problem hiding this comment.
Avoid the version-36 Kubernetes wheel regression
The previous dependency update (283cdf26) tested Kubernetes 36.0.2 and reverted it because its wheel grew from 1.92 MB to 4.40 MB, adding about 2.48 MB and tripping the Agent static-size quality gate. Moving to the adjacent 36.0.3 release without any corresponding size mitigation reintroduces the same version-36 generated-client payload into the resolve-build-deps.yaml artifacts, preventing the dependency set from passing that gate.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
More details
The updated ClickHouse, Redis, and Kubernetes client pins preserved connection construction, Redis fallback/error paths, and Kueue API-version fallback behavior in freshly resolved environments. No additional tests recommended: production telemetry is unavailable and the exercised compatibility branches are already covered by deterministic unit scenarios.
📊 Validated against 336 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit 9e58751 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
What does this PR do?
Update the dependencies
Dependency Bumps
This PR was automatically generated by the following workflow:
https://github.com/DataDog/integrations-core/actions/runs/30233414530
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.