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

fix ambiguous redis connection config #2543

Merged
merged 1 commit into from
Oct 2, 2024
Merged

Commits on Sep 24, 2024

  1. fix ambiguous redis connection config

    Redis sentinel is what's deployed in S3C, to connect to it
    we use "sentinels" field in redis config that contains the host/port
    of each sentinel.
    
    In Artesca we use the "host" and "port" fields to connect to the redis
    instance.
    
    The config rules were adapted to make sure we don't pass all the fields at once
    which might make the config ambiguous to understand, although the redis client
    seems to handle this case well, as it first checks the sentinel config before connecting
    using the host/port fields.
    
    Default values for redis host/port were removed from the config definition and put inside
    the default/dev config file.
    
    Issue: BB-522
    Kerkesni committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    d20c311 View commit details
    Browse the repository at this point in the history