salt module - redis - Adding multiples instances in minion conf
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 15.7k
- Forks
- 5.6k
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 80
Description
Is your feature request related to a problem? Please describe.
I have an host with several redis instances, the issue is that apparently we can set only a redis instance in minion configuration.
Describe the solution you'd like
Solution can be add multiple instances with a tag like that:
instance_1:
redis.host: 'salt'
redis.port: 6379
redis.db: 0
redis.password: XXX
instance_2:
redis.host: 'salt'
redis.port: 6380
redis.db: 0
redis.password: XXX
Once done, the idea is to launch directly salt like "salt '*' redis.bgrewriteaof instance=instance_1" by example.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue identifies the Redis Salt module and minion configuration as the starting points; first inspect how the current single Redis instance is read and how redis.bgrewriteaof arguments are dispatched. Define the named-instance configuration and instance selection behavior, then add coverage showing both instances can be targeted.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, redis
- Domain
- devops
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100