-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Ignoring unrecognized environment variable REGISTRY_REDIS_ADDRS #4621
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
Comments
See #4486 (comment) for a workaround |
@afunix that works for me: It looks like the environment variable parsing mechanism does not respect the yaml inline directive for |
Thanks for the workaround. I hope it will be fixed with documentation or in the code. |
For those of you who stumbled here for docker-compose/ environment:
REGISTRY_STORAGE_CACHE_BLOBDESCRIPTOR: redis
REGISTRY_REDIS_OPTIONS_ADDRS: "[redis:6379]" |
Description
I’ve migrated to registry v3 since it is officially out to have Redis Sentinel support.
Configuration format for Redis changed a bit, but the convention in environment variables naming is still the same in v3 (according to https://github.com/distribution/distribution/blob/main/docs/content/about/configuration.md).
I’m setting a number of env vars to configure the registry (among others REGISTRY_STORAGE_CACHE_BLOBDESCRIPTOR, REGISTRY_STORAGE_FILESYSTEM_ROOTDIRECTORY, etc), however all Redis-related variables are reported as unknown:
Is there a bug in documentation? Is there a bug in Redis configuration?
Reproduce
start registry v3
Expected behavior
registry successfully connects to Redis Sentinel
registry version
registry github.com/distribution/distribution/v3 3.0.0
Additional Info
No response
The text was updated successfully, but these errors were encountered: