search

AmadeusITGroup / Redis-Operator

Redis Operator creates/configures/manages Redis clusters atop Kubernetes
MIT License
167 stars 62 forks source link

Redis-Node configuration: bind 0.0.0.0 only #40

Closed clamoriniere1A closed 5 years ago

clamoriniere1A commented 5 years ago

Issues: #39

Previous implementation configure the redis-server process to bind all IPs present at redis-node pods start time. In some specific network configuration (like describe in #39). an static IP is added asynchronously after start time. Which leads to a connection issue from this added IP. Fixe consists in configuring the redis-server process to accept any connection.

codecov-io commented 5 years ago

Codecov Report

Merging #40 into master will decrease coverage by 0.01%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   36.88%   36.86%   -0.02%     
==========================================
  Files          37       37              
  Lines        3427     3423       -4     
==========================================
- Hits         1264     1262       -2     
+ Misses       2040     2039       -1     
+ Partials      123      122       -1
Impacted Files Coverage Δ
pkg/redisnode/node.go 51.21% <0%> (+0.05%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bcb86bf...9e3a333. Read the comment docs.