Update dependencies#24527
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a3d2ec1658
ℹ️ 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 RESP2 for Redis 5
This bump crosses redis-py 8.0, whose maintainers note that clients now use RESP3 by default and that this causes connection failures for Redis <6 or proxies without HELLO support (release note, issue); our Redis check still tests/supports Redis 5.0 in redisdb/hatch.toml and _get_conn constructs redis.Redis(**connection_params) without any protocol=2 override, so Redis 5 users will fail the handshake and report redis.can_connect critical instead of collecting metrics. Please either stay on 7.x or force/expose RESP2 before taking this major version.
Useful? React with 👍 / 👎.
Validation ReportAll 21 validations passed. Show details
|
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/29220602063
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.