Skip to content

Commit 0889c3e

Browse files
authored
Merge pull request kubernetes#75761 from fenggw-fnst/mybranch
Fix comments about node health monitor
2 parents c3bec0b + 5714041 commit 0889c3e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

pkg/controller/nodelifecycle/node_lifecycle_controller.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -623,8 +623,10 @@ func (nc *Controller) doEvictionPass() {
623623
}
624624

625625
// monitorNodeHealth verifies node health are constantly updated by kubelet, and
626-
// if not, post "NodeReady==ConditionUnknown". It also evicts all pods if node
627-
// is not ready or not reachable for a long period of time.
626+
// if not, post "NodeReady==ConditionUnknown".
627+
// For nodes who are not ready or not reachable for a long period of time.
628+
// This function will taint them if TaintBasedEvictions feature was enabled.
629+
// Otherwise, it would evict it directly.
628630
func (nc *Controller) monitorNodeHealth() error {
629631
// We are listing nodes from local cache as we can tolerate some small delays
630632
// comparing to state from etcd and there is eventual consistency anyway.

0 commit comments

Comments
 (0)