-
Notifications
You must be signed in to change notification settings - Fork 80
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
Add envinronment namespace as part of redis queue name for services #7217
Comments
Hello @vr-varad , thanks for offering help with this one. To be honest this is a time sensitive issue that we added directly to the current sprint and a developer is already picking it up. But if you are interested in contributing I can definitely point you out to some issues. best regards |
Ya sure @txau I am ready. |
@vr-varad here are some issues that are less time sensitive and sort of low hanging fruit: |
@gabriel-piles I assume this is fixed with #7246 and can be closed, correct? |
@aphilop yes, it is working well in staging, we can close it |
At the moment our queue names for certain services are hard coded. This is mixing the redis messages when using the same redis server and services for two different environments. We agreed with the ML team that in order to be able to use the same services from different environments we should add the environment name as prefix or suffix to the queue name.
The text was updated successfully, but these errors were encountered: