-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat/ Redis HA fix #283
Feat/ Redis HA fix #283
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question, what are now the restrictions to integrate with Redis? A specific version?
Should we fallback to the previous functionality if "leader-host" is not there? Otherwise, should we tell the user that this charm requires a Redis charm revision greater than X?
Ok, that is a great question. I did not think of that. I think I should ask @gregory-schiano about this. I think the fallback to unit data if |
Test coverage for 8aa08d6
Static code analysis report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Includes the Redis HA fix. This fixes 2 different bugs:
1- https://bugs.launchpad.net/juju/+bug/2063087
2- #268
Overview
Started using app databag for Redis hostname. Add tests to test out Redis in HA mode.
Rationale
Previously was not able to use Redis in HA mode
Juju Events Changes
Module Changes
Library Changes
Checklist
src-docs
urgent
,trivial
,complex
)