123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527 |
- /*
- 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 v1alpha1
- import (
- "testing"
- "github.com/google/go-cmp/cmp"
- "k8s.io/apimachinery/pkg/conversion"
- "k8s.io/apimachinery/pkg/runtime"
- componentbaseconfig "k8s.io/component-base/config"
- componentbaseconfigv1alpha1 "k8s.io/component-base/config/v1alpha1"
- "k8s.io/kube-scheduler/config/v1alpha1"
- "k8s.io/kubernetes/pkg/scheduler/apis/config"
- "k8s.io/utils/pointer"
- )
- func TestConvertKubeSchedulerConfiguration(t *testing.T) {
- cases := []struct {
- name string
- cfg v1alpha1.KubeSchedulerConfiguration
- want config.KubeSchedulerConfiguration
- }{
- {
- name: "scheduler name",
- cfg: v1alpha1.KubeSchedulerConfiguration{
- SchedulerName: pointer.StringPtr("custom-name"),
- },
- want: config.KubeSchedulerConfiguration{
- Profiles: []config.KubeSchedulerProfile{
- {SchedulerName: "custom-name"},
- },
- },
- },
- {
- name: "plugins and plugin config",
- cfg: v1alpha1.KubeSchedulerConfiguration{
- Plugins: &v1alpha1.Plugins{
- QueueSort: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "FooPlugin"},
- },
- },
- },
- PluginConfig: []v1alpha1.PluginConfig{
- {Name: "FooPlugin"},
- },
- },
- want: config.KubeSchedulerConfiguration{
- Profiles: []config.KubeSchedulerProfile{
- {
- Plugins: &config.Plugins{
- QueueSort: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "FooPlugin"},
- },
- },
- },
- PluginConfig: []config.PluginConfig{
- {Name: "FooPlugin"},
- },
- },
- },
- },
- },
- {
- name: "custom hard pod affinity weight",
- cfg: v1alpha1.KubeSchedulerConfiguration{
- HardPodAffinitySymmetricWeight: pointer.Int32Ptr(3),
- },
- want: config.KubeSchedulerConfiguration{
- Profiles: []config.KubeSchedulerProfile{
- {
- PluginConfig: []config.PluginConfig{
- {
- Name: "InterPodAffinity",
- Args: runtime.Unknown{
- Raw: []byte(`{"hardPodAffinityWeight":3}`),
- },
- },
- },
- },
- },
- },
- },
- {
- name: "custom hard pod affinity weight and existing PluginConfig",
- cfg: v1alpha1.KubeSchedulerConfiguration{
- HardPodAffinitySymmetricWeight: pointer.Int32Ptr(3),
- PluginConfig: []v1alpha1.PluginConfig{
- {
- Name: "InterPodAffinity",
- Args: runtime.Unknown{
- Raw: []byte(`{"hardPodAffinityWeight":5}`),
- },
- },
- },
- },
- want: config.KubeSchedulerConfiguration{
- Profiles: []config.KubeSchedulerProfile{
- {
- PluginConfig: []config.PluginConfig{
- {
- Name: "InterPodAffinity",
- Args: runtime.Unknown{
- Raw: []byte(`{"hardPodAffinityWeight":5}`),
- },
- },
- {
- Name: "InterPodAffinity",
- Args: runtime.Unknown{
- Raw: []byte(`{"hardPodAffinityWeight":3}`),
- },
- },
- },
- },
- },
- },
- },
- }
- scheme := getScheme(t)
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- var out config.KubeSchedulerConfiguration
- err := scheme.Convert(&tc.cfg, &out, nil)
- if err != nil {
- t.Fatal(err)
- }
- if diff := cmp.Diff(tc.want, out); diff != "" {
- t.Errorf("unexpected conversion (-want, +got):\n%s", diff)
- }
- })
- }
- }
- func TestV1alpha1ToConfigKubeSchedulerLeaderElectionConfiguration(t *testing.T) {
- configuration := &v1alpha1.KubeSchedulerLeaderElectionConfiguration{
- LockObjectName: "name",
- LockObjectNamespace: "namespace",
- LeaderElectionConfiguration: componentbaseconfigv1alpha1.LeaderElectionConfiguration{
- ResourceName: "name",
- ResourceNamespace: "namespace",
- },
- }
- emptyLockObjectNameConfig := configuration.DeepCopy()
- emptyLockObjectNameConfig.LockObjectName = ""
- emptyLockObjectNamespaceConfig := configuration.DeepCopy()
- emptyLockObjectNamespaceConfig.LockObjectNamespace = ""
- emptyResourceNameConfig := configuration.DeepCopy()
- emptyResourceNameConfig.ResourceName = ""
- emptyResourceNamespaceConfig := configuration.DeepCopy()
- emptyResourceNamespaceConfig.ResourceNamespace = ""
- differentNameConfig := configuration.DeepCopy()
- differentNameConfig.LockObjectName = "name1"
- differentNamespaceConfig := configuration.DeepCopy()
- differentNamespaceConfig.LockObjectNamespace = "namespace1"
- emptyconfig := &v1alpha1.KubeSchedulerLeaderElectionConfiguration{}
- scenarios := map[string]struct {
- expectedResourceNamespace string
- expectedResourceName string
- expectedToFailed bool
- config *v1alpha1.KubeSchedulerLeaderElectionConfiguration
- }{
- "both-set-same-name-and-namespace": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedToFailed: false,
- config: configuration,
- },
- "not-set-lock-object-name": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedToFailed: false,
- config: emptyLockObjectNameConfig,
- },
- "not-set-lock-object-namespace": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedToFailed: false,
- config: emptyLockObjectNamespaceConfig,
- },
- "not-set-resource-name": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedToFailed: false,
- config: emptyResourceNameConfig,
- },
- "not-set-resource-namespace": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedToFailed: false,
- config: emptyResourceNamespaceConfig,
- },
- "set-different-name": {
- expectedResourceNamespace: "",
- expectedResourceName: "",
- expectedToFailed: true,
- config: differentNameConfig,
- },
- "set-different-namespace": {
- expectedResourceNamespace: "",
- expectedResourceName: "",
- expectedToFailed: true,
- config: differentNamespaceConfig,
- },
- "set-empty-name-and-namespace": {
- expectedResourceNamespace: "",
- expectedResourceName: "",
- expectedToFailed: false,
- config: emptyconfig,
- },
- }
- for name, scenario := range scenarios {
- out := &config.KubeSchedulerLeaderElectionConfiguration{}
- s := conversion.Scope(nil)
- err := Convert_v1alpha1_KubeSchedulerLeaderElectionConfiguration_To_config_KubeSchedulerLeaderElectionConfiguration(scenario.config, out, s)
- if err == nil && scenario.expectedToFailed {
- t.Errorf("Unexpected success for scenario: %s", name)
- }
- if err == nil && !scenario.expectedToFailed {
- if out.ResourceName != scenario.expectedResourceName {
- t.Errorf("Unexpected success for scenario: %s, out.ResourceName: %s, expectedResourceName: %s", name, out.ResourceName, scenario.expectedResourceName)
- }
- if out.ResourceNamespace != scenario.expectedResourceNamespace {
- t.Errorf("Unexpected success for scenario: %s, out.ResourceNamespace: %s, expectedResourceNamespace: %s", name, out.ResourceNamespace, scenario.expectedResourceNamespace)
- }
- }
- if err != nil && !scenario.expectedToFailed {
- t.Errorf("Unexpected failure for scenario: %s - %+v", name, err)
- }
- }
- }
- func TestConfigToV1alpha1KubeSchedulerLeaderElectionConfiguration(t *testing.T) {
- configuration := &config.KubeSchedulerLeaderElectionConfiguration{
- LeaderElectionConfiguration: componentbaseconfig.LeaderElectionConfiguration{
- ResourceName: "name",
- ResourceNamespace: "namespace",
- },
- }
- emptyconfig := &config.KubeSchedulerLeaderElectionConfiguration{}
- scenarios := map[string]struct {
- expectedResourceNamespace string
- expectedResourceName string
- expectedLockObjectNamespace string
- expectedLockObjectName string
- expectedToFailed bool
- config *config.KubeSchedulerLeaderElectionConfiguration
- }{
- "both-set-name-and-namespace": {
- expectedResourceNamespace: "namespace",
- expectedResourceName: "name",
- expectedLockObjectNamespace: "namespace",
- expectedLockObjectName: "name",
- expectedToFailed: false,
- config: configuration,
- },
- "set-empty-name-and-namespace": {
- expectedResourceNamespace: "",
- expectedResourceName: "",
- expectedLockObjectNamespace: "",
- expectedLockObjectName: "",
- expectedToFailed: false,
- config: emptyconfig,
- },
- }
- for name, scenario := range scenarios {
- out := &v1alpha1.KubeSchedulerLeaderElectionConfiguration{}
- s := conversion.Scope(nil)
- err := Convert_config_KubeSchedulerLeaderElectionConfiguration_To_v1alpha1_KubeSchedulerLeaderElectionConfiguration(scenario.config, out, s)
- if err == nil && scenario.expectedToFailed {
- t.Errorf("Unexpected success for scenario: %s", name)
- }
- if err == nil && !scenario.expectedToFailed {
- if out.ResourceName != scenario.expectedResourceName {
- t.Errorf("Unexpected success for scenario: %s, out.ResourceName: %s, expectedResourceName: %s", name, out.ResourceName, scenario.expectedResourceName)
- }
- if out.LockObjectName != scenario.expectedLockObjectName {
- t.Errorf("Unexpected success for scenario: %s, out.LockObjectName: %s, expectedLockObjectName: %s", name, out.LockObjectName, scenario.expectedLockObjectName)
- }
- if out.ResourceNamespace != scenario.expectedResourceNamespace {
- t.Errorf("Unexpected success for scenario: %s, out.ResourceNamespace: %s, expectedResourceNamespace: %s", name, out.ResourceNamespace, scenario.expectedResourceNamespace)
- }
- if out.LockObjectNamespace != scenario.expectedLockObjectNamespace {
- t.Errorf("Unexpected success for scenario: %s, out.LockObjectNamespace: %s, expectedLockObjectNamespace: %s", name, out.LockObjectNamespace, scenario.expectedLockObjectNamespace)
- }
- }
- if err != nil && !scenario.expectedToFailed {
- t.Errorf("Unexpected failure for scenario: %s - %+v", name, err)
- }
- }
- }
- func TestConvertBetweenV1Alpha1PluginsAndConfigPlugins(t *testing.T) {
- // weight is assigned to score plugins
- weight := int32(10)
- // DummyWeight is a placeholder for the v1alpha1.plugins' weight will be filled with zero when
- // convert back from config.
- dummyWeight := int32(42)
- v1alpha1Plugins := v1alpha1.Plugins{
- QueueSort: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "queuesort-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-queuesort-plugin", Weight: &dummyWeight},
- },
- },
- PreFilter: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "prefilter-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-prefilter-plugin", Weight: &dummyWeight},
- },
- },
- Filter: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "filter-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-filter-plugin", Weight: &dummyWeight},
- },
- },
- PostFilter: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "postfilter-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-postfilter-plugin", Weight: &dummyWeight},
- },
- },
- Score: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "score-plugin", Weight: &weight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-score-plugin", Weight: &weight},
- },
- },
- Reserve: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "reserve-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-reserve-plugin", Weight: &dummyWeight},
- },
- },
- Permit: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "permit-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-permit-plugin", Weight: &dummyWeight},
- },
- },
- PreBind: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "prebind-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-prebind-plugin", Weight: &dummyWeight},
- },
- },
- Bind: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "bind-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-bind-plugin", Weight: &dummyWeight},
- },
- },
- PostBind: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "postbind-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-postbind-plugin", Weight: &dummyWeight},
- },
- },
- Unreserve: &v1alpha1.PluginSet{
- Enabled: []v1alpha1.Plugin{
- {Name: "unreserve-plugin", Weight: &dummyWeight},
- },
- Disabled: []v1alpha1.Plugin{
- {Name: "disabled-unreserve-plugin", Weight: &dummyWeight},
- },
- },
- }
- configPlugins := config.Plugins{
- QueueSort: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "queuesort-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-queuesort-plugin", Weight: dummyWeight},
- },
- },
- PreFilter: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "prefilter-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-prefilter-plugin", Weight: dummyWeight},
- },
- },
- Filter: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "filter-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-filter-plugin", Weight: dummyWeight},
- },
- },
- PreScore: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "postfilter-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-postfilter-plugin", Weight: dummyWeight},
- },
- },
- Score: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "score-plugin", Weight: weight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-score-plugin", Weight: weight},
- },
- },
- Reserve: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "reserve-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-reserve-plugin", Weight: dummyWeight},
- },
- },
- Permit: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "permit-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-permit-plugin", Weight: dummyWeight},
- },
- },
- PreBind: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "prebind-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-prebind-plugin", Weight: dummyWeight},
- },
- },
- Bind: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "bind-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-bind-plugin", Weight: dummyWeight},
- },
- },
- PostBind: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "postbind-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-postbind-plugin", Weight: dummyWeight},
- },
- },
- Unreserve: &config.PluginSet{
- Enabled: []config.Plugin{
- {Name: "unreserve-plugin", Weight: dummyWeight},
- },
- Disabled: []config.Plugin{
- {Name: "disabled-unreserve-plugin", Weight: dummyWeight},
- },
- },
- }
- convertedConfigPlugins := config.Plugins{}
- convertedV1Alpha1Plugins := v1alpha1.Plugins{}
- scheme := getScheme(t)
- if err := scheme.Convert(&v1alpha1Plugins, &convertedConfigPlugins, nil); err != nil {
- t.Fatal(err)
- }
- if err := scheme.Convert(&configPlugins, &convertedV1Alpha1Plugins, nil); err != nil {
- t.Fatal(err)
- }
- if diff := cmp.Diff(configPlugins, convertedConfigPlugins); diff != "" {
- t.Errorf("unexpected plugins diff (-want, +got): %s", diff)
- }
- if diff := cmp.Diff(v1alpha1Plugins, convertedV1Alpha1Plugins); diff != "" {
- t.Errorf("unexpected plugins diff (-want, +got): %s", diff)
- }
- }
- func getScheme(t *testing.T) *runtime.Scheme {
- scheme := runtime.NewScheme()
- if err := AddToScheme(scheme); err != nil {
- t.Fatal(err)
- }
- return scheme
- }
|