Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(elasticache): open one port, switch to valkey #116

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

hannahhoward
Copy link
Member

Goals

Noticing high connect times for redis in traces -- probably just first query but still https://docs.aws.amazon.com/AmazonElastiCache/latest/dg/wwe-troubleshooting.html recommends having both ports open in case a client tries to connect to both

Also, we need to switch to valkey before we turn it on cause it's almost 1/3 cheaper per unit.

Implementation

  • open both ports
  • upgrade AWS provider to get valkey support
  • move to valkey
  • set a dummy password on the disabled default user -- no-password-required apparently not supported for valkey
  • fix some downstream effects of create_before_destroy being applied at top of stack (shouldn't be as create_before_destroy will not work for many resources)

Copy link

codecov bot commented Feb 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 1 file with indirect coverage changes

@volmedo
Copy link
Member

volmedo commented Feb 11, 2025

tested this in my staging env and everything looks good, merging.

PS: I see a similar pattern of the first query to the cache being slow, it looks like it's due to warm up indeed.

@volmedo volmedo merged commit 844f180 into main Feb 11, 2025
9 checks passed
@volmedo volmedo deleted the feat/valkey-switch-and-one-more-port branch February 11, 2025 12:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants