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: upgrade redis-ha chart to 4.22.3, redis regression #11176
Conversation
Signed-off-by: Justin Marquis <34fathombelow@protonmail.com>
Codecov ReportBase: 45.60% // Head: 45.60% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #11176 +/- ##
=======================================
Coverage 45.60% 45.60%
=======================================
Files 237 237
Lines 28914 28914
=======================================
Hits 13185 13185
Misses 13913 13913
Partials 1816 1816
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Justin Marquis <34fathombelow@protonmail.com>
Signed-off-by: Justin Marquis <34fathombelow@protonmail.com>
Tested a sample of this fix on slack and worked well on our eks cluster. |
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
* chore: upgrade redis-ha chart to 4.22.3, redis regression Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix manifest Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix missing cidr Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix typo Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix typo Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> Signed-off-by: Justin Marquis <34fathombelow@protonmail.com>
Cherry-picked onto release-2.5 for 2.5.2. |
Glad to see a fix for this! Well done 👏 👏 |
* chore: upgrade redis-ha chart to 4.22.3, redis regression Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix manifest Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix missing cidr Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix typo Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> * fix typo Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> Signed-off-by: Justin Marquis <34fathombelow@protonmail.com> Signed-off-by: emirot <emirot.nolan@gmail.com>
Signed-off-by: Justin Marquis 34fathombelow@protonmail.com
Supersedes #11127
Fixes #11126
Fixes split-brain-fix problem associated with pods going into a crashing loop. Also allows redis pods to make DNS queries outside the cluster.
Special thanks for the awesome people that provided reporting, logs, analysis, and testing.