Skip to content

Commit

Permalink
Limit Redis dependencies to match celery limits (#33773)
Browse files Browse the repository at this point in the history
Redis 5 relased last week breaks celery, celery is limiting it for
now and will resolve it later, we should similarly limit redis on
our side to limit redis for users who will not upgrade to celery
that will be released shortly.

Fixes: #33744
  • Loading branch information
potiuk authored Aug 26, 2023
1 parent 42bc8fc commit 3ba994d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion airflow/providers/redis/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,11 @@ versions:

dependencies:
- apache-airflow>=2.4.0
- redis>=3.2.0
# We limit redis to <5.0.0 because of incompatibility with celery. Both Celery and Kombu limited it
# and deferred fixing it for later, we should bump the limit once they do. Also !=4.5.5 matches celery
# limits and prevents installing 4.5.5 which is broken.
# https://github.com/celery/celery/pull/8442, https://github.com/celery/kombu/pull/1776
- redis>=4.5.2,<5.0.0,!=4.5.5

integrations:
- integration-name: Redis
Expand Down
2 changes: 1 addition & 1 deletion generated/provider_dependencies.json
Original file line number Diff line number Diff line change
Expand Up @@ -754,7 +754,7 @@
"redis": {
"deps": [
"apache-airflow>=2.4.0",
"redis>=3.2.0"
"redis>=4.5.2,<5.0.0,!=4.5.5"
],
"cross-providers-deps": [],
"excluded-python-versions": []
Expand Down

0 comments on commit 3ba994d

Please sign in to comment.