github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-28 14:07:14 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #37293 from wojtek-t/fix_scheduler_bug

Browse Source 
Fix a bug in scheduler happening after retrying unsuccessful bindings
This commit is contained in:
Dawn Chen 2016-11-23 13:20:12 -08:00 committed by GitHub
commit ead3cc8c48
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
plugin/pkg/scheduler/scheduler.go
View File

@ -114,6 +114,13 @@ func (s *Scheduler) scheduleOne() {
assumed.Spec.NodeName = dest assumed.Spec.NodeName = dest
if err := s.config.SchedulerCache.AssumePod(&assumed); err != nil { if err := s.config.SchedulerCache.AssumePod(&assumed); err != nil {
glog.Errorf("scheduler cache AssumePod failed: %v", err) glog.Errorf("scheduler cache AssumePod failed: %v", err)
// TODO: This means that a given pod is already in cache (which means it
// is either assumed or already added). This is most probably result of a
// BUG in retrying logic. As a temporary workaround (which doesn't fully
// fix the problem, but should reduce its impact), we simply return here,
// as binding doesn't make sense anyway.
// This should be fixed properly though.
return
} }
go func() { go func() {