-
Couldn't load subscription status.
- Fork 0
feat: replace redis-cluster chart with redis-ha #38
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
Conversation
Co-authored-by: merll <merll@users.noreply.github.com>
|
During startup I see the following of our provided Kyverno policies fail, where we should aim to eliminate as many as possible:
One more thing I wonder if we should reconsider, since we are making a change here anyway: Currently we sometimes attach the product name to the service, which especially in test scenarios can lead to awkward naming like So my suggestion is to name the service just like the workload, e.g. the user calls it |
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.
restore.ssh.image.tag needs to be encapsulated with ""
📌 Summary
This PR replaces bitnami redis-cluster chart with redis-ha (https://github.com/DandyDeveloper/charts/tree/master/charts/redis-ha)
Ticket: https://track.akamai.com/jira/browse/APL-1079
PRs: apl-api | apl-docs
🔍 Reviewer Notes
Testing:
g6-dedicated-8to avoid hitting resource limits),or update the
otomi-apiapp _rawValues in your cluster:team-admin, navigate to the Catalog page in the console and add a newredis-haHelm chart using thisChart.yamllink: https://github.com/linode/apl-charts/blob/APL-1079/redis-ha/Chart.yamlteam-demo)