Only schedule to pods that are available. #5545

plugin/pkg/scheduler/factory/factory.go

@@ -230,8 +230,8 @@ func (factory *ConfigFactory) pollMinions() (cache.Enumerator, error) {
 			}
 		} else {
 			// If no condition is set, we get unknown node condition. In such cases,
-			// we add nodes unconditionally.
-			nodes.Items = append(nodes.Items, node)
+			// do not add the node
+			glog.V(2).Infof("Minion %s is not available.  Skipping", node.Name)
 		}
 	}
 	return &nodeEnumerator{nodes}, nil

plugin/pkg/scheduler/factory/factory_test.go

@@ -220,7 +220,7 @@ func TestPollMinions(t *testing.T) {
 					},
 				},
 			},
-			expectedCount: 6,
+			expectedCount: 5,
 		},
 		{
 			minions: []api.Node{
@@ -262,7 +262,7 @@ func TestPollMinions(t *testing.T) {
 					},
 				},
 			},
-			expectedCount: 1,
+			expectedCount: 0,
 		},
 		{
 			minions: []api.Node{
@@ -299,7 +299,7 @@ func TestPollMinions(t *testing.T) {
 					},
 				},
 			},
-			expectedCount: 1,
+			expectedCount: 0,
 		},
 	}