-
Notifications
You must be signed in to change notification settings - Fork 568
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
The documentation makes it sound like persistence is disabled by default #262
Comments
Is there a question, suggestion, or request in here?
(This sounds like a statement to me, and I'm not understanding what you're
hoping we'll do with it.)
|
An issue :) Suggestion to improve the documentation. Not a question. The documentation is misleading about this. I'm not sure if this part is needed at all. Although maybe in older version of I'm not sure where that part of the documentation comes from. If it covers only the latest builds, then one option is to remove the section. Or suggest to run https://github.com/docker-library/redis/blob/f634377/6.0/alpine/Dockerfile#L40-L48 |
I just confirmed that |
@yosifkit It totally looks that way, but on closer inspection it appears that It can be cured by explicitly specifying the #!/bin/sh -eu
run() {
cid=`docker run --rm -itd redis:alpine "$@"`
docker exec "$cid" redis-cli config get save
docker stop "$cid"
}
run
run --bind 0.0.0.0
run --save 3600 1 --save 300 100 --save 60 10000 Output:
|
This is confusing. I want to run with no persistence, so I've appendonly and save in the config file, but a volume is still being created. Not the end of the world but it's not clear from the docs how to disable this. |
+1 to this, there's room for clarification. https://hub.docker.com/r/redis/redis-stack is where I am coming from In its persistence section, it documents what happens when passing |
https://hub.docker.com/_/redis
The text was updated successfully, but these errors were encountered: