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

[FBZ-10466] Custom-Redis cookbook added #121

Open
wants to merge 2 commits into
base: next-release
Choose a base branch
from

Conversation

pasanchamikara
Copy link
Contributor

Description of your patch

This adds the custom-redis cookbook which depends on ey-redis. This would help the clients to configure redis on solo environments and to setup slave feature for redis instance.

Recommended Release Notes

Adds Customizations of Redis installation based on a custom-cookbook. For the time being, this adds the capacity to run redis on a more customized manner (i.e. On solo instances, add slave host feature, etc.)

Estimated risk

Medium

Components involved

custom-redis
ey-redis

Dependencies

ey-redis

Description of testing done

Booted a solo environment and validated installation of Redis on a solo environment
Booted a multi environment and validated the installation of Redis on master-slave manner on a multi environment

QA Instructions

  • On a solo environment
  1. Boot a solo instance
  2. Copy the contents of the custom-redis/cookbooks/ to an empty directory of choice
  3. Enable installation of redis on solo environment (Solo instance precisely), and re-upload the recipes
  4. See if redis is installed by /etc/init.d/redis-server status
  • On a multi environment
  1. Boot a multi environment with 2 util instances util1 and redis
  2. Set the redis slave as util1 as per the readme.
  3. Upload and re-apply the recipes
  4. Check by redis-cli info | grep ^role if the roles are properly set.

@pasanchamikara pasanchamikara self-assigned this Oct 7, 2022
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.

1 participant