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.

Sign up for GitHub

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

Jump to bottom

Builds are failing on Travis for Django 2.x because of redis-py 3 #159

Closed
gregsadetsky opened this issue Dec 7, 2018 · 0 comments · Fixed by #161
Closed

Builds are failing on Travis for Django 2.x because of redis-py 3 #159

gregsadetsky opened this issue Dec 7, 2018 · 0 comments · Fixed by #161

Comments

@gregsadetsky
Copy link

The most recent build fails for all DJANGO_VERSION values of 2.0 and 2.1. The error message for all 5 failing jobs is the same:

"redis.exceptions.DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first."

The issue is documented here. A suggested fix is to pin redis==2.10.6 in requirements.txt.
jsocol added a commit that referenced this issue Dec 29, 2018
@jsocol
Pin Redis version for tests
de986da 
Attempt to fix #159 by pinning the redis version to <3 for compatibility
with django-redis.
jsocol added a commit that referenced this issue Dec 29, 2018
@jsocol
Pin Redis version for tests
614171d 
Attempt to fix #159 by pinning the redis version to <3 for compatibility
with django-redis.
This was referenced Dec 29, 2018
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pin Redis version for tests jsocol/django-ratelimit
1 participant
@gregsadetsky