/* Copyright 2019 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 noderesources import ( "context" "testing" "github.com/google/go-cmp/cmp" v1 "k8s.io/api/core/v1" "k8s.io/apimachinery/pkg/util/validation/field" "k8s.io/klog/v2/ktesting" "k8s.io/kubernetes/pkg/scheduler/apis/config" "k8s.io/kubernetes/pkg/scheduler/framework" plfeature "k8s.io/kubernetes/pkg/scheduler/framework/plugins/feature" "k8s.io/kubernetes/pkg/scheduler/framework/runtime" "k8s.io/kubernetes/pkg/scheduler/internal/cache" st "k8s.io/kubernetes/pkg/scheduler/testing" tf "k8s.io/kubernetes/pkg/scheduler/testing/framework" ) func TestLeastAllocatedScoringStrategy(t *testing.T) { tests := []struct { name string requestedPod *v1.Pod nodes []*v1.Node existingPods []*v1.Pod expectedScores framework.NodeScoreList resources []config.ResourceSpec wantErrs field.ErrorList wantStatusCode framework.Code }{ { // Node1 scores (remaining resources) on 0-MaxNodeScore scale // CPU Score: ((4000 - 0) * MaxNodeScore) / 4000 = MaxNodeScore // Memory Score: ((10000 - 0) * MaxNodeScore) / 10000 = MaxNodeScore // Node1 Score: (100 + 100) / 2 = 100 // Node2 scores (remaining resources) on 0-MaxNodeScore scale // CPU Score: ((4000 - 0) * MaxNodeScore) / 4000 = MaxNodeScore // Memory Score: ((10000 - 0) * MaxNodeScore) / 10000 = MaxNodeScore // Node2 Score: (MaxNodeScore + MaxNodeScore) / 2 = MaxNodeScore name: "nothing scheduled, nothing requested", requestedPod: st.MakePod().Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: framework.MaxNodeScore}, {Name: "node2", Score: framework.MaxNodeScore}}, resources: defaultResources, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((4000 - 3000) * MaxNodeScore) / 4000 = 25 // Memory Score: ((10000 - 5000) * MaxNodeScore) / 10000 = 50 // Node1 Score: (25 + 50) / 2 = 37 // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((6000 - 3000) * MaxNodeScore) / 6000 = 50 // Memory Score: ((10000 - 5000) * MaxNodeScore) / 10000 = 50 // Node2 Score: (50 + 50) / 2 = 50 name: "nothing scheduled, resources requested, differently sized nodes", requestedPod: st.MakePod(). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: 37}, {Name: "node2", Score: 50}}, resources: defaultResources, }, { name: "Resources not set, pods scheduled with error", requestedPod: st.MakePod(). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: framework.MinNodeScore}, {Name: "node2", Score: framework.MinNodeScore}}, resources: nil, wantStatusCode: framework.Error, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((4000 - 0) * MaxNodeScore) / 4000 = MaxNodeScore // Memory Score: ((10000 - 0) * MaxNodeScore) / 10000 = MaxNodeScore // Node1 Score: (MaxNodeScore + MaxNodeScore) / 2 = MaxNodeScore // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((4000 - 0) * MaxNodeScore) / 4000 = MaxNodeScore // Memory Score: ((10000 - 0) * MaxNodeScore) / 10000 = MaxNodeScore // Node2 Score: (MaxNodeScore + MaxNodeScore) / 2 = MaxNodeScore name: "no resources requested, pods scheduled", requestedPod: st.MakePod().Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Obj(), st.MakePod().Node("node1").Obj(), st.MakePod().Node("node2").Obj(), st.MakePod().Node("node2").Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: framework.MaxNodeScore}, {Name: "node2", Score: framework.MaxNodeScore}}, resources: defaultResources, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((20000 - 0) * MaxNodeScore) / 20000 = MaxNodeScore // Node1 Score: (40 + 100) / 2 = 70 // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((20000 - 5000) * MaxNodeScore) / 20000 = 75 // Node2 Score: (40 + 75) / 2 = 57 name: "no resources requested, pods scheduled with resources", requestedPod: st.MakePod().Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "20000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "20000"}).Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 70}, {Name: "node2", Score: 57}}, resources: defaultResources, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((20000 - 5000) * MaxNodeScore) / 20000 = 75 // Node1 Score: (40 + 75) / 2 = 57 // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((20000 - 10000) * MaxNodeScore) / 20000 = 50 // Node2 Score: (40 + 50) / 2 = 45 name: "resources requested, pods scheduled with resources", requestedPod: st.MakePod(). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "20000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "20000"}).Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 57}, {Name: "node2", Score: 45}}, resources: defaultResources, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((20000 - 5000) * MaxNodeScore) / 20000 = 75 // Node1 Score: (40 + 75) / 2 = 57 // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((10000 - 6000) * MaxNodeScore) / 10000 = 40 // Memory Score: ((50000 - 10000) * MaxNodeScore) / 50000 = 80 // Node2 Score: (40 + 80) / 2 = 60 name: "resources requested, pods scheduled with resources, differently sized nodes", requestedPod: st.MakePod(). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "20000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "10000", "memory": "50000"}).Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 57}, {Name: "node2", Score: 60}}, resources: defaultResources, }, { // Node1 scores on 0-MaxNodeScore scale // CPU Score: ((4000 - 6000) * MaxNodeScore) / 4000 = 0 // Memory Score: ((10000 - 0) * MaxNodeScore) / 10000 = MaxNodeScore // Node1 Score: (0 + MaxNodeScore) / 2 = 50 // Node2 scores on 0-MaxNodeScore scale // CPU Score: ((4000 - 6000) * MaxNodeScore) / 4000 = 0 // Memory Score: ((10000 - 5000) * MaxNodeScore) / 10000 = 50 // Node2 Score: (0 + 50) / 2 = 25 name: "requested resources exceed node capacity", requestedPod: st.MakePod().Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 50}, {Name: "node2", Score: 25}}, resources: defaultResources, }, { name: "zero node resources, pods scheduled with resources", requestedPod: st.MakePod().Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Obj(), st.MakeNode().Name("node2").Obj(), }, existingPods: []*v1.Pod{ st.MakePod().Node("node1").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "0"}).Obj(), st.MakePod().Node("node2").Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: framework.MinNodeScore}, {Name: "node2", Score: framework.MinNodeScore}}, resources: defaultResources, }, { // CPU Score: ((4000 - 3000) *100) / 4000 = 25 // Memory Score: ((10000 - 5000) *100) / 10000 = 50 // Node1 Score: (25 * 1 + 50 * 2) / (1 + 2) = 41 // CPU Score: ((6000 - 3000) *100) / 6000 = 50 // Memory Score: ((10000 - 5000) *100) / 10000 = 50 // Node2 Score: (50 * 1 + 50 * 2) / (1 + 2) = 50 name: "nothing scheduled, resources requested with different weight on CPU and memory, differently sized nodes", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: 41}, {Name: "node2", Score: 50}}, resources: []config.ResourceSpec{ {Name: "memory", Weight: 2}, {Name: "cpu", Weight: 1}, }, }, { // resource with negative weight is not allowed name: "resource with negative weight", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, resources: []config.ResourceSpec{ {Name: "memory", Weight: -1}, {Name: "cpu", Weight: 1}, }, wantErrs: field.ErrorList{ &field.Error{ Type: field.ErrorTypeInvalid, Field: "scoringStrategy.resources[0].weight", }, }, }, { // resource with zero weight is not allowed name: "resource with zero weight", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: 41}, {Name: "node2", Score: 50}}, resources: []config.ResourceSpec{ {Name: "memory", Weight: 1}, {Name: "cpu", Weight: 0}, }, wantErrs: field.ErrorList{ &field.Error{ Type: field.ErrorTypeInvalid, Field: "scoringStrategy.resources[1].weight", }, }, }, { // resource weight should be less than MaxNodeScore name: "resource weight larger than MaxNodeScore", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "4000", "memory": "10000"}).Obj(), }, resources: []config.ResourceSpec{ {Name: "memory", Weight: 1}, {Name: "cpu", Weight: 101}, }, wantErrs: field.ErrorList{ &field.Error{ Type: field.ErrorTypeInvalid, Field: "scoringStrategy.resources[1].weight", }, }, }, { // Bypass extended resource if the pod does not request. // For both nodes: cpuScore and memScore are 50 // Given that extended resource score are intentionally bypassed, // the final scores are: // - node1: (50 + 50) / 2 = 50 // - node2: (50 + 50) / 2 = 50 name: "bypass extended resource if the pod does not request", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "1000", "memory": "2000"}). Req(map[v1.ResourceName]string{"cpu": "2000", "memory": "3000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000", v1.ResourceName(extendedRes): "4"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 50}, {Name: "node2", Score: 50}}, resources: extendedResourceSet, }, { // Honor extended resource if the pod requests. // For both nodes: cpuScore and memScore are 50. // In terms of extended resource score: // - node1 get: 2 / 4 * 100 = 50 // - node2 get: (10 - 2) / 10 * 100 = 80 // So the final scores are: // - node1: (50 + 50 + 50) / 3 = 50 // - node2: (50 + 50 + 80) / 3 = 60 name: "honor extended resource if the pod requests", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "5000", v1.ResourceName(extendedRes): "2"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000", v1.ResourceName(extendedRes): "4"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000", v1.ResourceName(extendedRes): "10"}).Obj(), }, existingPods: nil, expectedScores: []framework.NodeScore{{Name: "node1", Score: 50}, {Name: "node2", Score: 60}}, resources: extendedResourceSet, }, { // If the node doesn't have a resource // CPU Score: ((6000 - 3000) * 100) / 6000 = 50 // Memory Score: ((10000 - 4000) * 100) / 10000 = 60 // Node1 Score: (50 * 1 + 60 * 1) / (1 + 1) = 55 // Node2 Score: (50 * 1 + 60 * 1) / (1 + 1) = 55 name: "if the node doesn't have a resource", requestedPod: st.MakePod().Node("node1"). Req(map[v1.ResourceName]string{"cpu": "3000", "memory": "4000"}). Obj(), nodes: []*v1.Node{ st.MakeNode().Name("node1").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000"}).Obj(), st.MakeNode().Name("node2").Capacity(map[v1.ResourceName]string{"cpu": "6000", "memory": "10000", v1.ResourceName(extendedRes): "4"}).Obj(), }, expectedScores: []framework.NodeScore{{Name: "node1", Score: 55}, {Name: "node2", Score: 55}}, resources: []config.ResourceSpec{ {Name: extendedRes, Weight: 2}, {Name: string(v1.ResourceCPU), Weight: 1}, {Name: string(v1.ResourceMemory), Weight: 1}, }, }, } for _, test := range tests { t.Run(test.name, func(t *testing.T) { _, ctx := ktesting.NewTestContext(t) ctx, cancel := context.WithCancel(ctx) defer cancel() state := framework.NewCycleState() snapshot := cache.NewSnapshot(test.existingPods, test.nodes) fh, _ := runtime.NewFramework(ctx, nil, nil, runtime.WithSnapshotSharedLister(snapshot)) p, err := NewFit( ctx, &config.NodeResourcesFitArgs{ ScoringStrategy: &config.ScoringStrategy{ Type: config.LeastAllocated, Resources: test.resources, }, }, fh, plfeature.Features{}) if diff := cmp.Diff(test.wantErrs.ToAggregate(), err, ignoreBadValueDetail); diff != "" { t.Fatalf("got err (-want,+got):\n%s", diff) } if err != nil { return } status := p.(framework.PreScorePlugin).PreScore(ctx, state, test.requestedPod, tf.BuildNodeInfos(test.nodes)) if !status.IsSuccess() { t.Errorf("PreScore is expected to return success, but didn't. Got status: %v", status) } var gotScores framework.NodeScoreList for _, n := range test.nodes { score, status := p.(framework.ScorePlugin).Score(ctx, state, test.requestedPod, n.Name) if status.Code() != test.wantStatusCode { t.Errorf("unexpected status code, want: %v, got: %v", test.wantStatusCode, status.Code()) } gotScores = append(gotScores, framework.NodeScore{Name: n.Name, Score: score}) } if diff := cmp.Diff(test.expectedScores, gotScores); diff != "" { t.Errorf("Unexpected scores (-want,+got):\n%s", diff) } }) } }