From 1ae12337dbad7a923c82346871079da1fcd98b39 Mon Sep 17 00:00:00 2001 From: Mayank Kumar Date: Sun, 16 Sep 2018 23:49:15 -0700 Subject: [PATCH] scheduler: add error handling for bind --- pkg/scheduler/scheduler.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/scheduler/scheduler.go b/pkg/scheduler/scheduler.go index d140e40b1c..aca2b9f666 100644 --- a/pkg/scheduler/scheduler.go +++ b/pkg/scheduler/scheduler.go @@ -369,8 +369,8 @@ func (sched *Scheduler) bind(assumed *v1.Pod, b *v1.Binding) error { // If binding succeeded then PodScheduled condition will be updated in apiserver so that // it's atomic with setting host. err := sched.config.GetBinder(assumed).Bind(b) - if err := sched.config.SchedulerCache.FinishBinding(assumed); err != nil { - glog.Errorf("scheduler cache FinishBinding failed: %v", err) + if finErr := sched.config.SchedulerCache.FinishBinding(assumed); finErr != nil { + glog.Errorf("scheduler cache FinishBinding failed: %v", finErr) } if err != nil { glog.V(1).Infof("Failed to bind pod: %v/%v", assumed.Namespace, assumed.Name)