Enable default redis instance #988
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The default redis instance should be automatically enabled/started after install, but it does not always seem to happen.
It looks like it was missed in this clean up : openfoodfoundation@28193ef
4 tasks
dacook
approved these changes
Jan 15, 2025
There was a problem hiding this comment.
Great, although it seems like something that should be handled by the galaxy role.
It might be easy enough to contribute: https://github.com/libre-ops/multi_redis/blob/main/tasks/main.yml
But for now, this does the job 👍
mkllnk
approved these changes
Jan 16, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While looking at setting monitoring for all redis instances, I noticed a some of our prod instances don't have the default redis instances. That means we can't update the ActionCable configuration to use said instance as proposed here openfoodfoundation/openfoodnetwork#13072
According to the documentation redis should be enable/start automatically but it appears to not always be the case.
This PR add a new role to make sure the default instance of redis is enable and running when provisioning a server.