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

Redis health check is not sending the auth command on it's connection #8166

Merged
merged 13 commits into from
Sep 11, 2019
Merged

Redis health check is not sending the auth command on it's connection #8166

merged 13 commits into from
Sep 11, 2019

Commits on Sep 6, 2019

  1. Refactor redis auth call to redis client factory. 

    Make the redis health checker also call auth if auth password is configured.

Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    662ebe4 View commit details
    Browse the repository at this point in the history

  2. fix format 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    fc133da View commit details
    Browse the repository at this point in the history

  3. Add release note. 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    7a2f64d View commit details
    Browse the repository at this point in the history

  4. Fix failed tests 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    36591d6 View commit details
    Browse the repository at this point in the history

  5. fix format 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    4898669 View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2019

  1. Avoid READONLY commands for the connections used for CLUSTER SLOTS. 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 9, 2019
    Configuration menu
    Copy the full SHA
    bc29429 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into health_check_auth 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 9, 2019
    Configuration menu
    Copy the full SHA
    d8d7217 View commit details
    Browse the repository at this point in the history

  3. Fix merge issue. 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 9, 2019
    Configuration menu
    Copy the full SHA
    4c8ab45 View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2019

  1. Fix review feedback and cache auth_password used in redis cluster 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 10, 2019
    Configuration menu
    Copy the full SHA
    6531a42 View commit details
    Browse the repository at this point in the history

  2. Fix format. 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 10, 2019
    Configuration menu
    Copy the full SHA
    73d02c8 View commit details
    Browse the repository at this point in the history

Commits on Sep 11, 2019

  1. Merge branch 'master' into health_check_auth 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 11, 2019
    Configuration menu
    Copy the full SHA
    a656dfa View commit details
    Browse the repository at this point in the history

  2. Fix format 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 11, 2019
    Configuration menu
    Copy the full SHA
    92cd158 View commit details
    Browse the repository at this point in the history

  3. Fix merge issue. 

    Signed-off-by: Henry Yang <hyang@lyft.com>
    @HenryYYang
    HenryYYang committed Sep 11, 2019
    Configuration menu
    Copy the full SHA
    ff82723 View commit details
    Browse the repository at this point in the history