Merge pull request #91294 from cofyc/fix90962-donot-change-pod-in-assume-phase

SchedulerVolumeBinder: don't modify assumed pod in AssumePodVolumes
This commit is contained in:
Kubernetes Prow Robot 2020-05-20 19:42:00 -07:00 committed by GitHub
commit 0d6db3dfdd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -115,7 +115,6 @@ type SchedulerVolumeBinder interface {
//
// It returns true if all volumes are fully bound
//
// This function will modify assumedPod with the node name.
// This function is called serially.
AssumePodVolumes(assumedPod *v1.Pod, nodeName string) (allFullyBound bool, err error)
@ -334,8 +333,6 @@ func (b *volumeBinder) AssumePodVolumes(assumedPod *v1.Pod, nodeName string) (al
return true, nil
}
assumedPod.Spec.NodeName = nodeName
claimsToBind := b.podBindingCache.GetBindings(assumedPod, nodeName)
claimsToProvision := b.podBindingCache.GetProvisionedPVCs(assumedPod, nodeName)