sous-chefs / redisio

Development repository for the redisio cookbook
https://supermarket.chef.io/cookbooks/redisio
Apache License 2.0
297 stars 301 forks source link

Missing required sentinel parameter master_ip #449

Closed damacus closed 3 years ago

damacus commented 3 years ago

:ghost: Brief Description

Missing required sentinel parameter master_ip # In /opt/kitchen/cache/cookbooks/redisio/recipes/sentinel.rb 24: redisio_sentinel 'redis-sentinels' do 25: version redis['version'] if redis['version'] 26: sentinel_defaults redis['sentinel_defaults'] 27: sentinels sentinel_instances 28: base_piddir redis['base_piddir'] 29: end 30: Compiled Resource: ------------------ # Declared in /opt/kitchen/cache/cookbooks/redisio/recipes/sentinel.rb:24:in `from_file' redisio_sentinel("redis-sentinels") do action [:run] default_guard_interpreter :default declared_type :redisio_sentinel cookbook_name "redisio" recipe_name "sentinel" version "3.2.11" sentinel_defaults {"user"=>"redis", "configdir"=>"/etc/redis", "sentinel_bind"=>nil, "sentinel_port"=>26379, "monitor"=>nil, "down_after_milliseconds"=>30000, "can-failover"=>"yes", "parallel-syncs"=>1, "failover_timeout"=>900000, "loglevel"=>"notice", "logfile"=>nil, "syslogenabled"=>"yes", "syslogfacility"=>"local0", "quorum_count"=>2, "data_bag_name"=>nil, "data_bag_item"=>nil, "data_bag_key"=>nil, "announce-ip"=>nil, "announce-port"=>nil, "notification-script"=>nil, "client-reconfig-script"=>nil} sentinels [{"name"=>"cluster", "sentinel_bind"=>"0.0.0.0", "sentinel_port"=>26379, "masters"=>nil}, {"name"=>"sentinel6379", "master_name"=>"master6379", "master_ip"=>"127.0.0.1", "master_port"=>6379}, {"name"=>"sentinel6380", "master_name"=>"master6380", "master_ip"=>"127.0.0.1", "master_port"=>6380}] base_piddir "/var/run/redis" user "redis" end

:pancakes: Cookbook version

main

:woman_cook: Chef-Infra Version

17.4

:tophat: Platform details

CentOS 7

Steps To Reproduce

kitchen test multisentinel-debian-9

:police_car: Expected behaviour

Multisentinel test pass

:heavy_plus_sign: Additional context

Happens on this PR