openstack
/
kuryr-kubernetes
Code Issues Proposed changes

Fix health server binding address 

Flask update changed the default binding address from 0.0.0.0 to
127.0.0.1. This affects Kuryr's healthchecks as we were depending on
that behavior. This manifested with Kuryr pods being constantly
restarted.

This commit fixes that by explicitly binding to 0.0.0.0.

Change-Id: I298960c2e84befe74c35b32bcc6730ba27202604
Related-Bug: 1778516
Closes-Bug: 1778867
This commit is contained in:
Michał Dulko 2018-06-27 11:06:05 +02:00
parent 1829623fbd
commit a3e9f6a869
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
kuryr_kubernetes/cni/health.py
View File

@ -152,7 +152,7 @@ class CNIHealthServer(object):
return data, httplib.OK, self.headers
def run(self):
address = ''
address = '0.0.0.0'
try:
LOG.info('Starting CNI health check server.')
self.application.run(address, CONF.cni_health_server.port)

2
kuryr_kubernetes/controller/managers/health.py
View File

@ -103,7 +103,7 @@ class HealthServer(object):
return data, httplib.OK, self.headers
def run(self):
address = ''
address = '0.0.0.0'
try:
LOG.info('Starting health check server.')
self.application.run(address, CONF.health_server.port)