123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374 |
- /*
- Copyright 2014 The Kubernetes Authors.
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
- http://www.apache.org/licenses/LICENSE-2.0
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
- package scheduler
- import (
- "fmt"
- "math"
- "reflect"
- "strconv"
- "testing"
- "time"
- "k8s.io/kubernetes/pkg/api"
- "k8s.io/kubernetes/pkg/util/sets"
- "k8s.io/kubernetes/pkg/util/wait"
- "k8s.io/kubernetes/plugin/pkg/scheduler/algorithm"
- algorithmpredicates "k8s.io/kubernetes/plugin/pkg/scheduler/algorithm/predicates"
- schedulerapi "k8s.io/kubernetes/plugin/pkg/scheduler/api"
- "k8s.io/kubernetes/plugin/pkg/scheduler/schedulercache"
- )
- func falsePredicate(pod *api.Pod, meta interface{}, nodeInfo *schedulercache.NodeInfo) (bool, []algorithm.PredicateFailureReason, error) {
- return false, []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate}, nil
- }
- func truePredicate(pod *api.Pod, meta interface{}, nodeInfo *schedulercache.NodeInfo) (bool, []algorithm.PredicateFailureReason, error) {
- return true, nil, nil
- }
- func matchesPredicate(pod *api.Pod, meta interface{}, nodeInfo *schedulercache.NodeInfo) (bool, []algorithm.PredicateFailureReason, error) {
- node := nodeInfo.Node()
- if node == nil {
- return false, nil, fmt.Errorf("node not found")
- }
- if pod.Name == node.Name {
- return true, nil, nil
- }
- return false, []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate}, nil
- }
- func hasNoPodsPredicate(pod *api.Pod, meta interface{}, nodeInfo *schedulercache.NodeInfo) (bool, []algorithm.PredicateFailureReason, error) {
- if len(nodeInfo.Pods()) == 0 {
- return true, nil, nil
- }
- return false, []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate}, nil
- }
- func numericPriority(pod *api.Pod, nodeNameToInfo map[string]*schedulercache.NodeInfo, nodes []*api.Node) (schedulerapi.HostPriorityList, error) {
- result := []schedulerapi.HostPriority{}
- for _, node := range nodes {
- score, err := strconv.Atoi(node.Name)
- if err != nil {
- return nil, err
- }
- result = append(result, schedulerapi.HostPriority{
- Host: node.Name,
- Score: score,
- })
- }
- return result, nil
- }
- func reverseNumericPriority(pod *api.Pod, nodeNameToInfo map[string]*schedulercache.NodeInfo, nodes []*api.Node) (schedulerapi.HostPriorityList, error) {
- var maxScore float64
- minScore := math.MaxFloat64
- reverseResult := []schedulerapi.HostPriority{}
- result, err := numericPriority(pod, nodeNameToInfo, nodes)
- if err != nil {
- return nil, err
- }
- for _, hostPriority := range result {
- maxScore = math.Max(maxScore, float64(hostPriority.Score))
- minScore = math.Min(minScore, float64(hostPriority.Score))
- }
- for _, hostPriority := range result {
- reverseResult = append(reverseResult, schedulerapi.HostPriority{
- Host: hostPriority.Host,
- Score: int(maxScore + minScore - float64(hostPriority.Score)),
- })
- }
- return reverseResult, nil
- }
- func makeNodeList(nodeNames []string) []*api.Node {
- result := make([]*api.Node, 0, len(nodeNames))
- for _, nodeName := range nodeNames {
- result = append(result, &api.Node{ObjectMeta: api.ObjectMeta{Name: nodeName}})
- }
- return result
- }
- func TestSelectHost(t *testing.T) {
- scheduler := genericScheduler{}
- tests := []struct {
- list schedulerapi.HostPriorityList
- possibleHosts sets.String
- expectsErr bool
- }{
- {
- list: []schedulerapi.HostPriority{
- {Host: "machine1.1", Score: 1},
- {Host: "machine2.1", Score: 2},
- },
- possibleHosts: sets.NewString("machine2.1"),
- expectsErr: false,
- },
- // equal scores
- {
- list: []schedulerapi.HostPriority{
- {Host: "machine1.1", Score: 1},
- {Host: "machine1.2", Score: 2},
- {Host: "machine1.3", Score: 2},
- {Host: "machine2.1", Score: 2},
- },
- possibleHosts: sets.NewString("machine1.2", "machine1.3", "machine2.1"),
- expectsErr: false,
- },
- // out of order scores
- {
- list: []schedulerapi.HostPriority{
- {Host: "machine1.1", Score: 3},
- {Host: "machine1.2", Score: 3},
- {Host: "machine2.1", Score: 2},
- {Host: "machine3.1", Score: 1},
- {Host: "machine1.3", Score: 3},
- },
- possibleHosts: sets.NewString("machine1.1", "machine1.2", "machine1.3"),
- expectsErr: false,
- },
- // empty priorityList
- {
- list: []schedulerapi.HostPriority{},
- possibleHosts: sets.NewString(),
- expectsErr: true,
- },
- }
- for _, test := range tests {
- // increase the randomness
- for i := 0; i < 10; i++ {
- got, err := scheduler.selectHost(test.list)
- if test.expectsErr {
- if err == nil {
- t.Error("Unexpected non-error")
- }
- } else {
- if err != nil {
- t.Errorf("Unexpected error: %v", err)
- }
- if !test.possibleHosts.Has(got) {
- t.Errorf("got %s is not in the possible map %v", got, test.possibleHosts)
- }
- }
- }
- }
- }
- func TestGenericScheduler(t *testing.T) {
- tests := []struct {
- name string
- predicates map[string]algorithm.FitPredicate
- prioritizers []algorithm.PriorityConfig
- nodes []string
- pod *api.Pod
- pods []*api.Pod
- expectedHosts sets.String
- expectsErr bool
- wErr error
- }{
- {
- predicates: map[string]algorithm.FitPredicate{"false": falsePredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: EqualPriority, Weight: 1}},
- nodes: []string{"machine1", "machine2"},
- expectsErr: true,
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- name: "test 1",
- wErr: &FitError{
- Pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- FailedPredicates: FailedPredicateMap{
- "machine1": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- "machine2": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- }},
- },
- {
- predicates: map[string]algorithm.FitPredicate{"true": truePredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: EqualPriority, Weight: 1}},
- nodes: []string{"machine1", "machine2"},
- expectedHosts: sets.NewString("machine1", "machine2"),
- name: "test 2",
- wErr: nil,
- },
- {
- // Fits on a machine where the pod ID matches the machine name
- predicates: map[string]algorithm.FitPredicate{"matches": matchesPredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: EqualPriority, Weight: 1}},
- nodes: []string{"machine1", "machine2"},
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "machine2"}},
- expectedHosts: sets.NewString("machine2"),
- name: "test 3",
- wErr: nil,
- },
- {
- predicates: map[string]algorithm.FitPredicate{"true": truePredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: numericPriority, Weight: 1}},
- nodes: []string{"3", "2", "1"},
- expectedHosts: sets.NewString("3"),
- name: "test 4",
- wErr: nil,
- },
- {
- predicates: map[string]algorithm.FitPredicate{"matches": matchesPredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: numericPriority, Weight: 1}},
- nodes: []string{"3", "2", "1"},
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- expectedHosts: sets.NewString("2"),
- name: "test 5",
- wErr: nil,
- },
- {
- predicates: map[string]algorithm.FitPredicate{"true": truePredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: numericPriority, Weight: 1}, {Function: reverseNumericPriority, Weight: 2}},
- nodes: []string{"3", "2", "1"},
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- expectedHosts: sets.NewString("1"),
- name: "test 6",
- wErr: nil,
- },
- {
- predicates: map[string]algorithm.FitPredicate{"true": truePredicate, "false": falsePredicate},
- prioritizers: []algorithm.PriorityConfig{{Function: numericPriority, Weight: 1}},
- nodes: []string{"3", "2", "1"},
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- expectsErr: true,
- name: "test 7",
- wErr: &FitError{
- Pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- FailedPredicates: FailedPredicateMap{
- "3": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- "2": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- "1": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- },
- },
- },
- {
- predicates: map[string]algorithm.FitPredicate{
- "nopods": hasNoPodsPredicate,
- "matches": matchesPredicate,
- },
- pods: []*api.Pod{
- {
- ObjectMeta: api.ObjectMeta{Name: "2"},
- Spec: api.PodSpec{
- NodeName: "2",
- },
- Status: api.PodStatus{
- Phase: api.PodRunning,
- },
- },
- },
- pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- prioritizers: []algorithm.PriorityConfig{{Function: numericPriority, Weight: 1}},
- nodes: []string{"1", "2"},
- expectsErr: true,
- name: "test 8",
- wErr: &FitError{
- Pod: &api.Pod{ObjectMeta: api.ObjectMeta{Name: "2"}},
- FailedPredicates: FailedPredicateMap{
- "1": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- "2": []algorithm.PredicateFailureReason{algorithmpredicates.ErrFakePredicate},
- },
- },
- },
- }
- for _, test := range tests {
- cache := schedulercache.New(time.Duration(0), wait.NeverStop)
- for _, pod := range test.pods {
- cache.AddPod(pod)
- }
- for _, name := range test.nodes {
- cache.AddNode(&api.Node{ObjectMeta: api.ObjectMeta{Name: name}})
- }
- scheduler := NewGenericScheduler(cache, test.predicates, test.prioritizers, []algorithm.SchedulerExtender{})
- machine, err := scheduler.Schedule(test.pod, algorithm.FakeNodeLister(makeNodeList(test.nodes)))
- if !reflect.DeepEqual(err, test.wErr) {
- t.Errorf("Failed : %s, Unexpected error: %v, expected: %v", test.name, err, test.wErr)
- }
- if test.expectedHosts != nil && !test.expectedHosts.Has(machine) {
- t.Errorf("Failed : %s, Expected: %s, got: %s", test.name, test.expectedHosts, machine)
- }
- }
- }
- func TestFindFitAllError(t *testing.T) {
- nodes := []string{"3", "2", "1"}
- predicates := map[string]algorithm.FitPredicate{"true": truePredicate, "false": falsePredicate}
- nodeNameToInfo := map[string]*schedulercache.NodeInfo{
- "3": schedulercache.NewNodeInfo(),
- "2": schedulercache.NewNodeInfo(),
- "1": schedulercache.NewNodeInfo(),
- }
- _, predicateMap, err := findNodesThatFit(&api.Pod{}, nodeNameToInfo, makeNodeList(nodes), predicates, nil)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if len(predicateMap) != len(nodes) {
- t.Errorf("unexpected failed predicate map: %v", predicateMap)
- }
- for _, node := range nodes {
- failures, found := predicateMap[node]
- if !found {
- t.Errorf("failed to find node: %s in %v", node, predicateMap)
- }
- if len(failures) != 1 || failures[0] != algorithmpredicates.ErrFakePredicate {
- t.Errorf("unexpected failures: %v", failures)
- }
- }
- }
- func TestFindFitSomeError(t *testing.T) {
- nodes := []string{"3", "2", "1"}
- predicates := map[string]algorithm.FitPredicate{"true": truePredicate, "match": matchesPredicate}
- pod := &api.Pod{ObjectMeta: api.ObjectMeta{Name: "1"}}
- nodeNameToInfo := map[string]*schedulercache.NodeInfo{
- "3": schedulercache.NewNodeInfo(),
- "2": schedulercache.NewNodeInfo(),
- "1": schedulercache.NewNodeInfo(pod),
- }
- for name := range nodeNameToInfo {
- nodeNameToInfo[name].SetNode(&api.Node{ObjectMeta: api.ObjectMeta{Name: name}})
- }
- _, predicateMap, err := findNodesThatFit(pod, nodeNameToInfo, makeNodeList(nodes), predicates, nil)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if len(predicateMap) != (len(nodes) - 1) {
- t.Errorf("unexpected failed predicate map: %v", predicateMap)
- }
- for _, node := range nodes {
- if node == pod.Name {
- continue
- }
- failures, found := predicateMap[node]
- if !found {
- t.Errorf("failed to find node: %s in %v", node, predicateMap)
- }
- if len(failures) != 1 || failures[0] != algorithmpredicates.ErrFakePredicate {
- t.Errorf("unexpected failures: %v", failures)
- }
- }
- }
|