From f685af32e276be9356bbb36660c7c04d3a9dad4a Mon Sep 17 00:00:00 2001 From: Yecheng Fu Date: Wed, 20 May 2020 21:18:43 +0800 Subject: [PATCH] SchedulerVolumeBinder: don't modify assumed pod in AssumePodVolumes --- pkg/controller/volume/scheduling/scheduler_binder.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/controller/volume/scheduling/scheduler_binder.go b/pkg/controller/volume/scheduling/scheduler_binder.go index 1d5aeda7397..a2e5c3bdcec 100644 --- a/pkg/controller/volume/scheduling/scheduler_binder.go +++ b/pkg/controller/volume/scheduling/scheduler_binder.go @@ -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)