Merge pull request #51603 from hzxuzhonghu/taint-controller-panic

Automatic merge from submit-queue (batch tested with PRs 51603, 51653)

fix taint controller panic

**What this PR does / why we need it**:

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #51586 

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
This commit is contained in:
Kubernetes Submit Queue 2017-09-06 02:55:09 -07:00 committed by GitHub
commit ecfee6a9c9

View File

@ -376,6 +376,7 @@ func NewNodeController(
}
if nc.runTaintManager {
nc.taintManager = scheduler.NewNoExecuteTaintManager(kubeClient)
nodeInformer.Informer().AddEventHandler(cache.ResourceEventHandlerFuncs{
AddFunc: util.CreateAddNodeHandler(func(node *v1.Node) error {
nc.taintManager.NodeUpdated(nil, node)
@ -390,7 +391,6 @@ func NewNodeController(
return nil
}),
})
nc.taintManager = scheduler.NewNoExecuteTaintManager(kubeClient)
}
if nc.taintNodeByCondition {