kubernetes
/
autoscaler
mirror of https://github.com/kubernetes/autoscaler.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2096 from frobware/fix-segv-in-updateReadinessStats 

Fix potential SEGV in updateReadinessStats
This commit is contained in:
Kubernetes Prow Robot 2019-06-11 09:00:24 -07:00 committed by GitHub
parent c4fc0d8c0d 91016a605a
commit dd89fb1385
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cluster-autoscaler/clusterstate/clusterstate.go
View File

@ -585,6 +585,10 @@ func (csr *ClusterStateRegistry) updateReadinessStats(currentTime time.Time) {
klog.Warningf("Failed to get nodegroup for %s: %v", unregistered.Node.Name, errNg) klog.Warningf("Failed to get nodegroup for %s: %v", unregistered.Node.Name, errNg)
continue continue
} }
if nodeGroup == nil {
klog.Warningf("Nodegroup is nil for %s", unregistered.Node.Name)
continue
}
perNgCopy := perNodeGroup[nodeGroup.Id()] perNgCopy := perNodeGroup[nodeGroup.Id()]
if unregistered.UnregisteredSince.Add(csr.config.MaxNodeProvisionTime).Before(currentTime) { if unregistered.UnregisteredSince.Add(csr.config.MaxNodeProvisionTime).Before(currentTime) {
perNgCopy.LongUnregistered++ perNgCopy.LongUnregistered++