Commit afef4b69 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #32070 from gmarek/nodecontroller

Automatic merge from submit-queue Sleep between NodeStatus update retries Just a thing I found when looking into other problems. This is pretty much no-risk change fixing wrong behavior. Do you think it should go in 1.4? @pwittrock
parents c79a07f2 bac603af
...@@ -73,6 +73,8 @@ const ( ...@@ -73,6 +73,8 @@ const (
evictionRateLimiterBurst = 1 evictionRateLimiterBurst = 1
// The amount of time the nodecontroller polls on the list nodes endpoint. // The amount of time the nodecontroller polls on the list nodes endpoint.
apiserverStartupGracePeriod = 10 * time.Minute apiserverStartupGracePeriod = 10 * time.Minute
// The amount of time the nodecontroller should sleep between retrying NodeStatus updates
retrySleepTime = 20 * time.Millisecond
) )
type zoneState string type zoneState string
...@@ -538,6 +540,7 @@ func (nc *NodeController) monitorNodeStatus() error { ...@@ -538,6 +540,7 @@ func (nc *NodeController) monitorNodeStatus() error {
glog.Errorf("Failed while getting a Node to retry updating NodeStatus. Probably Node %s was deleted.", name) glog.Errorf("Failed while getting a Node to retry updating NodeStatus. Probably Node %s was deleted.", name)
break break
} }
time.Sleep(retrySleepTime)
} }
if err != nil { if err != nil {
glog.Errorf("Update status of Node %v from NodeController exceeds retry count."+ glog.Errorf("Update status of Node %v from NodeController exceeds retry count."+
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment