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.

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

cookbok-for-elasticache-redis-fbz-10238 #225

Open
wants to merge 1 commit into
base: next-release
Choose a base branch
from

Conversation

kstych
Copy link
Contributor

@kstych kstych commented Mar 3, 2022

Description of your patch

PR adds cookbook to configure elasticache redis on EYC, the cookbook added is named "elasticache-redis" which can be configured using environment variables EY_ELASTICACHE_REDIS_ENABLED , EY_ELASTICACHE_REDIS_URL

the cookbook creates config/redis.yml file as output
FBZ : https://t3.fogbugz.com/f/cases/10238/Enable-apps-to-use-ElastiCache-Redis

Recommended Release Notes

New Recipe : elasticache-redis : adds ability to configure redis.yml from environment variables

Estimated risk

Low

Components involved

Environment Variables/Chef

Dependencies

NA

Description of testing done

Tested to verify recipe generates config/redis.yml with correct values as set in environment variables
further testing/implementation to be done for ZD #183191

QA Instructions

Recipe required environment variables set

  1. EY_ELASTICACHE_REDIS_ENABLED set to true
  2. EY_ELASTICACHE_REDIS_URL set to redis endpoint without port
  3. EY_REDIS_ENABLED should not be set to true

@kstych kstych requested a review from mushyy March 3, 2022 13:07
Copy link
Contributor

@rupikakapoor rupikakapoor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants