Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

kubelet: Close health port in kubelet bootstrap pods #173

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

surajssd
Copy link
Contributor

Add livenessProbe and readinessProbe to self hosted kubelet.

invidian
invidian previously approved these changes Feb 12, 2020
Copy link
Contributor

@invidian invidian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine.

@surajssd surajssd force-pushed the surajssd/close-health-port branch 3 times, most recently from 491cc3d to 9283a5e Compare February 13, 2020 14:40
@invidian invidian force-pushed the surajssd/close-health-port branch from 9283a5e to a4e4f9c Compare February 18, 2020 11:42
@invidian
Copy link
Contributor

Pushed to trigger CI.

iaguis
iaguis previously approved these changes Feb 18, 2020
@iaguis iaguis force-pushed the surajssd/close-health-port branch from a4e4f9c to c6ec3bf Compare February 18, 2020 16:51
* This port is not used in Lokomotive for anything, so rather close it.

Signed-off-by: Suraj Deshmukh <[email protected]>
@invidian invidian force-pushed the surajssd/close-health-port branch from c6ec3bf to 19e0b6e Compare February 19, 2020 13:12
@invidian
Copy link
Contributor

Resolved conflicts.

@invidian invidian force-pushed the surajssd/close-health-port branch from 19e0b6e to 729f822 Compare February 19, 2020 13:16
@invidian
Copy link
Contributor

Kubelet listens on 127.0.0.1 by default, which I'm not sure is right for health checks, so let's try with --healthz-bind-address=$(HOST_IP).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants