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.

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

JuiceFS cannot use redis v7 #3223

Closed
OrvilleQ opened this issue Feb 9, 2023 · 2 comments · Fixed by #3233
Closed

JuiceFS cannot use redis v7 #3223

OrvilleQ opened this issue Feb 9, 2023 · 2 comments · Fixed by #3233
Assignees
Labels
kind/feature New feature or request

Comments

@OrvilleQ
Copy link

OrvilleQ commented Feb 9, 2023

Related to redis/go-redis#2085, go-redis v8 is not compatible with redis v7.

Is there any plan to upgrade this dependencies to v9 to support redis v7?

@OrvilleQ OrvilleQ added the kind/bug Something isn't working label Feb 9, 2023
@zhijian-pro zhijian-pro self-assigned this Feb 10, 2023
@zhijian-pro
Copy link
Contributor

We need to test whether go-redis v9 works with redis v6.

@jacksgt
Copy link

jacksgt commented Feb 11, 2023

I believe I just ran into this exact issue. Installed JuiceFS v0.17.5 with Redis 7.0.0
Provisioning a volume and writing to it worked fine, but upon destroying I get:

# juicefs status redis://:[email protected]:6379/1
2023/02/11 21:03:10.560897 juicefs[47] <INFO>: Meta address: redis://:****@redis-metadata-master.juicefs.svc.cluster.local:6379/1 [interface.go:402]
2023/02/11 21:03:10.562675 juicefs[47] <INFO>: Ping redis: 38.755µs [redis.go:2888]
{
  "Setting": {
    "Name": "juicefs-example",
    "UUID": "a06362c5-c64a-4165-bbe3-1522ebb017ac",
    "Storage": "minio",
    "Bucket": "https://example.com/foobar",
    "AccessKey": "foobar",
    "SecretKey": "removed",
    "BlockSize": 4096,
    "Compression": "none",
    "KeyEncrypted": true,
    "TrashDays": 1,
    "MetaVersion": 1
  },
  "Sessions": []
}

# juicefs destroy redis://:[email protected]:6379/1 a06362c5-c64a-4165-bbe3-1522ebb017ac --force
2023/02/11 21:03:30.206832 juicefs[73] <INFO>: Meta address: redis://:****@redis-metadata-master.juicefs.svc.cluster.local:6379/1 [interface.go:402]
2023/02/11 21:03:30.209044 juicefs[73] <INFO>: Ping redis: 44.738µs [redis.go:2888]
Deleted objects count: 0                                 ⠋                                 
Deleted objects count: 0   
Deleted objects count: 0   
2023/02/11 21:03:30.231855 juicefs[73] <FATAL>: reset meta: ERR unknown command 'flushdb', with args beginning with:  [destroy.go:208]

For the moment, at least a big warning in the docs would be helpful.

@zhijian-pro zhijian-pro added kind/feature New feature or request and removed kind/bug Something isn't working labels Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants