123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367 |
- /*
- Copyright 2018 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 config
- import (
- "bytes"
- "io/ioutil"
- "os"
- "path/filepath"
- "reflect"
- "testing"
- "github.com/pmezard/go-difflib/difflib"
- "k8s.io/api/core/v1"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- kubeadmapiv1beta2 "k8s.io/kubernetes/cmd/kubeadm/app/apis/kubeadm/v1beta2"
- "k8s.io/kubernetes/cmd/kubeadm/app/constants"
- "sigs.k8s.io/yaml"
- )
- func diff(expected, actual []byte) string {
- // Write out the diff
- var diffBytes bytes.Buffer
- difflib.WriteUnifiedDiff(&diffBytes, difflib.UnifiedDiff{
- A: difflib.SplitLines(string(expected)),
- B: difflib.SplitLines(string(actual)),
- FromFile: "expected",
- ToFile: "actual",
- Context: 3,
- })
- return diffBytes.String()
- }
- func TestLoadInitConfigurationFromFile(t *testing.T) {
- // Create temp folder for the test case
- tmpdir, err := ioutil.TempDir("", "")
- if err != nil {
- t.Fatalf("Couldn't create tmpdir")
- }
- defer os.RemoveAll(tmpdir)
- // cfgFiles is in cluster_test.go
- var tests = []struct {
- name string
- fileContents []byte
- expectErr bool
- }{
- {
- name: "v1beta1.partial1",
- fileContents: cfgFiles["InitConfiguration_v1beta1"],
- },
- {
- name: "v1beta1.partial2",
- fileContents: cfgFiles["ClusterConfiguration_v1beta1"],
- },
- {
- name: "v1beta1.full",
- fileContents: bytes.Join([][]byte{
- cfgFiles["InitConfiguration_v1beta1"],
- cfgFiles["ClusterConfiguration_v1beta1"],
- cfgFiles["Kube-proxy_componentconfig"],
- cfgFiles["Kubelet_componentconfig"],
- }, []byte(constants.YAMLDocumentSeparator)),
- },
- {
- name: "v1beta2.partial1",
- fileContents: cfgFiles["InitConfiguration_v1beta2"],
- },
- {
- name: "v1beta2.partial2",
- fileContents: cfgFiles["ClusterConfiguration_v1beta2"],
- },
- {
- name: "v1beta2.full",
- fileContents: bytes.Join([][]byte{
- cfgFiles["InitConfiguration_v1beta2"],
- cfgFiles["ClusterConfiguration_v1beta2"],
- cfgFiles["Kube-proxy_componentconfig"],
- cfgFiles["Kubelet_componentconfig"],
- }, []byte(constants.YAMLDocumentSeparator)),
- },
- }
- for _, rt := range tests {
- t.Run(rt.name, func(t2 *testing.T) {
- cfgPath := filepath.Join(tmpdir, rt.name)
- err := ioutil.WriteFile(cfgPath, rt.fileContents, 0644)
- if err != nil {
- t.Errorf("Couldn't create file")
- return
- }
- obj, err := LoadInitConfigurationFromFile(cfgPath)
- if rt.expectErr {
- if err == nil {
- t.Error("Unexpected success")
- }
- } else {
- if err != nil {
- t.Errorf("Error reading file: %v", err)
- return
- }
- if obj == nil {
- t.Errorf("Unexpected nil return value")
- }
- }
- })
- }
- }
- /*
- func TestInitConfigurationMarshallingFromFile(t *testing.T) {
- controlPlaneV1beta1YAMLAbstracted := controlPlaneV1beta1YAML
- controlPlaneInternalYAMLAbstracted := controlPlaneInternalYAML
- controlPlaneDefaultedYAMLAbstracted := controlPlaneDefaultedYAML
- if runtime.GOOS != "linux" {
- controlPlaneV1beta1YAMLAbstracted = controlPlaneV1beta1YAMLNonLinux
- controlPlaneInternalYAMLAbstracted = controlPlaneInternalYAMLNonLinux
- controlPlaneDefaultedYAMLAbstracted = controlPlaneDefaultedYAMLNonLinux
- }
- var tests = []struct {
- name, in, out string
- groupVersion schema.GroupVersion
- expectedErr bool
- }{
- // These tests are reading one file, loading it using LoadInitConfigurationFromFile that all of kubeadm is using for unmarshal of our API types,
- // and then marshals the internal object to the expected groupVersion
- //{ // v1beta1 -> internal NB. test commented after changes required for upgrading to go v1.12
- // name: "v1beta1ToInternal",
- // in: controlPlaneV1beta1YAMLAbstracted,
- // out: controlPlaneInternalYAMLAbstracted,
- // groupVersion: kubeadm.SchemeGroupVersion,
- //},
- { // v1beta1 -> internal -> v1beta1
- name: "v1beta1Tov1beta1",
- in: controlPlaneV1beta1YAMLAbstracted,
- out: controlPlaneV1beta1YAMLAbstracted,
- groupVersion: kubeadmapiv1beta1.SchemeGroupVersion,
- },
- // These tests are reading one file that has only a subset of the fields populated, loading it using LoadInitConfigurationFromFile,
- // and then marshals the internal object to the expected groupVersion
- { // v1beta1 -> default -> validate -> internal -> v1beta1
- name: "incompleteYAMLToDefaultedv1beta1",
- in: controlPlaneIncompleteYAML,
- out: controlPlaneDefaultedYAMLAbstracted,
- groupVersion: kubeadmapiv1beta1.SchemeGroupVersion,
- },
- { // v1beta1 -> validation should fail
- name: "invalidYAMLShouldFail",
- in: controlPlaneInvalidYAML,
- expectedErr: true,
- },
- }
- for _, rt := range tests {
- t.Run(rt.name, func(t2 *testing.T) {
- internalcfg, err := LoadInitConfigurationFromFile(rt.in)
- if err != nil {
- if rt.expectedErr {
- return
- }
- t2.Fatalf("couldn't unmarshal test data: %v", err)
- }
- actual, err := MarshalInitConfigurationToBytes(internalcfg, rt.groupVersion)
- if err != nil {
- t2.Fatalf("couldn't marshal internal object: %v", err)
- }
- expected, err := ioutil.ReadFile(rt.out)
- if err != nil {
- t2.Fatalf("couldn't read test data: %v", err)
- }
- if !bytes.Equal(expected, actual) {
- t2.Errorf("the expected and actual output differs.\n\tin: %s\n\tout: %s\n\tgroupversion: %s\n\tdiff: \n%s\n",
- rt.in, rt.out, rt.groupVersion.String(), diff(expected, actual))
- }
- })
- }
- }
- */
- func TestConsistentOrderByteSlice(t *testing.T) {
- var (
- aKind = "Akind"
- aFile = []byte(`
- kind: Akind
- apiVersion: foo.k8s.io/v1
- `)
- aaKind = "Aakind"
- aaFile = []byte(`
- kind: Aakind
- apiVersion: foo.k8s.io/v1
- `)
- abKind = "Abkind"
- abFile = []byte(`
- kind: Abkind
- apiVersion: foo.k8s.io/v1
- `)
- )
- var tests = []struct {
- name string
- in map[string][]byte
- expected [][]byte
- }{
- {
- name: "a_aa_ab",
- in: map[string][]byte{
- aKind: aFile,
- aaKind: aaFile,
- abKind: abFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- {
- name: "a_ab_aa",
- in: map[string][]byte{
- aKind: aFile,
- abKind: abFile,
- aaKind: aaFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- {
- name: "aa_a_ab",
- in: map[string][]byte{
- aaKind: aaFile,
- aKind: aFile,
- abKind: abFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- {
- name: "aa_ab_a",
- in: map[string][]byte{
- aaKind: aaFile,
- abKind: abFile,
- aKind: aFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- {
- name: "ab_a_aa",
- in: map[string][]byte{
- abKind: abFile,
- aKind: aFile,
- aaKind: aaFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- {
- name: "ab_aa_a",
- in: map[string][]byte{
- abKind: abFile,
- aaKind: aaFile,
- aKind: aFile,
- },
- expected: [][]byte{aaFile, abFile, aFile},
- },
- }
- for _, rt := range tests {
- t.Run(rt.name, func(t2 *testing.T) {
- actual := consistentOrderByteSlice(rt.in)
- if !reflect.DeepEqual(rt.expected, actual) {
- t2.Errorf("the expected and actual output differs.\n\texpected: %s\n\tout: %s\n", rt.expected, actual)
- }
- })
- }
- }
- func TestDefaultTaintsMarshaling(t *testing.T) {
- tests := []struct {
- desc string
- cfg kubeadmapiv1beta2.InitConfiguration
- expectedTaintCnt int
- }{
- {
- desc: "Uninitialized nodeRegistration field produces a single taint (the master one)",
- cfg: kubeadmapiv1beta2.InitConfiguration{
- TypeMeta: metav1.TypeMeta{
- APIVersion: "kubeadm.k8s.io/v1beta2",
- Kind: constants.InitConfigurationKind,
- },
- },
- expectedTaintCnt: 1,
- },
- {
- desc: "Uninitialized taints field produces a single taint (the master one)",
- cfg: kubeadmapiv1beta2.InitConfiguration{
- TypeMeta: metav1.TypeMeta{
- APIVersion: "kubeadm.k8s.io/v1beta2",
- Kind: constants.InitConfigurationKind,
- },
- NodeRegistration: kubeadmapiv1beta2.NodeRegistrationOptions{},
- },
- expectedTaintCnt: 1,
- },
- {
- desc: "Forsing taints to an empty slice produces no taints",
- cfg: kubeadmapiv1beta2.InitConfiguration{
- TypeMeta: metav1.TypeMeta{
- APIVersion: "kubeadm.k8s.io/v1beta2",
- Kind: constants.InitConfigurationKind,
- },
- NodeRegistration: kubeadmapiv1beta2.NodeRegistrationOptions{
- Taints: []v1.Taint{},
- },
- },
- expectedTaintCnt: 0,
- },
- {
- desc: "Custom taints are used",
- cfg: kubeadmapiv1beta2.InitConfiguration{
- TypeMeta: metav1.TypeMeta{
- APIVersion: "kubeadm.k8s.io/v1beta2",
- Kind: constants.InitConfigurationKind,
- },
- NodeRegistration: kubeadmapiv1beta2.NodeRegistrationOptions{
- Taints: []v1.Taint{
- {Key: "taint1"},
- {Key: "taint2"},
- },
- },
- },
- expectedTaintCnt: 2,
- },
- }
- for _, tc := range tests {
- t.Run(tc.desc, func(t *testing.T) {
- b, err := yaml.Marshal(tc.cfg)
- if err != nil {
- t.Fatalf("unexpected error while marshalling to YAML: %v", err)
- }
- cfg, err := BytesToInitConfiguration(b)
- if err != nil {
- t.Fatalf("unexpected error of BytesToInitConfiguration: %v\nconfig: %s", err, string(b))
- }
- if tc.expectedTaintCnt != len(cfg.NodeRegistration.Taints) {
- t.Fatalf("unexpected taints count\nexpected: %d\ngot: %d\ntaints: %v", tc.expectedTaintCnt, len(cfg.NodeRegistration.Taints), cfg.NodeRegistration.Taints)
- }
- })
- }
- }
|