Merge pull request #76059 from tedyu/master

Remove unnecessary map in cleanupHistory
This commit is contained in:
Kubernetes Prow Robot 2019-06-25 05:24:35 -07:00 committed by GitHub
commit a78ae4ba74
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -160,21 +160,13 @@ func (dsc *DaemonSetsController) cleanupHistory(ds *apps.DaemonSet, old []*apps.
}
}
// Find all live history with the above hashes
liveHistory := make(map[string]bool)
for _, history := range old {
if hash := history.Labels[apps.DefaultDaemonSetUniqueLabelKey]; liveHashes[hash] {
liveHistory[history.Name] = true
}
}
// Clean up old history from smallest to highest revision (from oldest to newest)
sort.Sort(historiesByRevision(old))
for _, history := range old {
if toKill <= 0 {
break
}
if liveHistory[history.Name] {
if hash := history.Labels[apps.DefaultDaemonSetUniqueLabelKey]; liveHashes[hash] {
continue
}
// Clean up