123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321 |
- /*
- 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 cmd
- import (
- "bytes"
- "encoding/json"
- "io/ioutil"
- "net/http"
- "reflect"
- "testing"
- "time"
- "k8s.io/kubernetes/pkg/api"
- "k8s.io/kubernetes/pkg/api/unversioned"
- "k8s.io/kubernetes/pkg/client/unversioned/fake"
- "k8s.io/kubernetes/pkg/conversion"
- cmdutil "k8s.io/kubernetes/pkg/kubectl/cmd/util"
- "k8s.io/kubernetes/pkg/runtime"
- )
- func generateNodeAndTaintedNode(oldTaints []api.Taint, newTaints []api.Taint) (*api.Node, *api.Node) {
- var taintedNode *api.Node
- oldTaintsData, _ := json.Marshal(oldTaints)
- // Create a node.
- node := &api.Node{
- ObjectMeta: api.ObjectMeta{
- Name: "node-name",
- CreationTimestamp: unversioned.Time{Time: time.Now()},
- Annotations: map[string]string{
- api.TaintsAnnotationKey: string(oldTaintsData),
- },
- },
- Spec: api.NodeSpec{
- ExternalID: "node-name",
- },
- Status: api.NodeStatus{},
- }
- clone, _ := conversion.NewCloner().DeepCopy(node)
- newTaintsData, _ := json.Marshal(newTaints)
- // A copy of the same node, but tainted.
- taintedNode = clone.(*api.Node)
- taintedNode.Annotations = map[string]string{
- api.TaintsAnnotationKey: string(newTaintsData),
- }
- return node, taintedNode
- }
- func AnnotationsHaveEqualTaints(annotationA map[string]string, annotationB map[string]string) bool {
- taintsA, err := api.GetTaintsFromNodeAnnotations(annotationA)
- if err != nil {
- return false
- }
- taintsB, err := api.GetTaintsFromNodeAnnotations(annotationB)
- if err != nil {
- return false
- }
- if len(taintsA) != len(taintsB) {
- return false
- }
- for _, taintA := range taintsA {
- found := false
- for _, taintB := range taintsB {
- if reflect.DeepEqual(taintA, taintB) {
- found = true
- break
- }
- }
- if !found {
- return false
- }
- }
- return true
- }
- func TestTaint(t *testing.T) {
- tests := []struct {
- description string
- oldTaints []api.Taint
- newTaints []api.Taint
- args []string
- expectFatal bool
- expectTaint bool
- }{
- // success cases
- {
- description: "taints a node with effect NoSchedule",
- newTaints: []api.Taint{{
- Key: "foo",
- Value: "bar",
- Effect: "NoSchedule",
- }},
- args: []string{"node", "node-name", "foo=bar:NoSchedule"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "taints a node with effect PreferNoSchedule",
- newTaints: []api.Taint{{
- Key: "foo",
- Value: "bar",
- Effect: "PreferNoSchedule",
- }},
- args: []string{"node", "node-name", "foo=bar:PreferNoSchedule"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "update an existing taint on the node, change the value from bar to barz",
- oldTaints: []api.Taint{{
- Key: "foo",
- Value: "bar",
- Effect: "NoSchedule",
- }},
- newTaints: []api.Taint{{
- Key: "foo",
- Value: "barz",
- Effect: "NoSchedule",
- }},
- args: []string{"node", "node-name", "foo=barz:NoSchedule", "--overwrite"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "taints a node with two taints",
- newTaints: []api.Taint{{
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "NoSchedule",
- }, {
- Key: "foo",
- Value: "bar",
- Effect: "PreferNoSchedule",
- }},
- args: []string{"node", "node-name", "dedicated=namespaceA:NoSchedule", "foo=bar:PreferNoSchedule"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "node has two taints with the same key but different effect, remove one of them by indicating exact key and effect",
- oldTaints: []api.Taint{{
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "NoSchedule",
- }, {
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "PreferNoSchedule",
- }},
- newTaints: []api.Taint{{
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "PreferNoSchedule",
- }},
- args: []string{"node", "node-name", "dedicated:NoSchedule-"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "node has two taints with the same key but different effect, remove all of them with wildcard",
- oldTaints: []api.Taint{{
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "NoSchedule",
- }, {
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "PreferNoSchedule",
- }},
- newTaints: []api.Taint{},
- args: []string{"node", "node-name", "dedicated-"},
- expectFatal: false,
- expectTaint: true,
- },
- {
- description: "node has two taints, update one of them and remove the other",
- oldTaints: []api.Taint{{
- Key: "dedicated",
- Value: "namespaceA",
- Effect: "NoSchedule",
- }, {
- Key: "foo",
- Value: "bar",
- Effect: "PreferNoSchedule",
- }},
- newTaints: []api.Taint{{
- Key: "foo",
- Value: "barz",
- Effect: "PreferNoSchedule",
- }},
- args: []string{"node", "node-name", "dedicated:NoSchedule-", "foo=barz:PreferNoSchedule", "--overwrite"},
- expectFatal: false,
- expectTaint: true,
- },
- // error cases
- {
- description: "invalid taint key",
- args: []string{"node", "node-name", "nospecialchars^@=banana:NoSchedule"},
- expectFatal: true,
- expectTaint: false,
- },
- {
- description: "invalid taint effect",
- args: []string{"node", "node-name", "foo=bar:NoExcute"},
- expectFatal: true,
- expectTaint: false,
- },
- {
- description: "duplicated taints with the same key and effect should be rejected",
- args: []string{"node", "node-name", "foo=bar:NoExcute", "foo=barz:NoExcute"},
- expectFatal: true,
- expectTaint: false,
- },
- {
- description: "can't update existing taint on the node, since 'overwrite' flag is not set",
- oldTaints: []api.Taint{{
- Key: "foo",
- Value: "bar",
- Effect: "NoSchedule",
- }},
- newTaints: []api.Taint{{
- Key: "foo",
- Value: "bar",
- Effect: "NoSchedule",
- }},
- args: []string{"node", "node-name", "foo=bar:NoSchedule"},
- expectFatal: true,
- expectTaint: false,
- },
- }
- for _, test := range tests {
- oldNode, expectNewNode := generateNodeAndTaintedNode(test.oldTaints, test.newTaints)
- new_node := &api.Node{}
- tainted := false
- f, tf, codec, ns := NewAPIFactory()
- tf.Client = &fake.RESTClient{
- NegotiatedSerializer: ns,
- Client: fake.CreateHTTPClient(func(req *http.Request) (*http.Response, error) {
- m := &MyReq{req}
- switch {
- case m.isFor("GET", "/nodes/node-name"):
- return &http.Response{StatusCode: 200, Header: defaultHeader(), Body: objBody(codec, oldNode)}, nil
- case m.isFor("PATCH", "/nodes/node-name"), m.isFor("PUT", "/nodes/node-name"):
- tainted = true
- data, err := ioutil.ReadAll(req.Body)
- if err != nil {
- t.Fatalf("%s: unexpected error: %v", test.description, err)
- }
- defer req.Body.Close()
- if err := runtime.DecodeInto(codec, data, new_node); err != nil {
- t.Fatalf("%s: unexpected error: %v", test.description, err)
- }
- if !AnnotationsHaveEqualTaints(expectNewNode.Annotations, new_node.Annotations) {
- t.Fatalf("%s: expected:\n%v\nsaw:\n%v\n", test.description, expectNewNode.Annotations, new_node.Annotations)
- }
- return &http.Response{StatusCode: 200, Header: defaultHeader(), Body: objBody(codec, new_node)}, nil
- default:
- t.Fatalf("%s: unexpected request: %v %#v\n%#v", test.description, req.Method, req.URL, req)
- return nil, nil
- }
- }),
- }
- tf.ClientConfig = defaultClientConfig()
- buf := bytes.NewBuffer([]byte{})
- cmd := NewCmdTaint(f, buf)
- saw_fatal := false
- func() {
- defer func() {
- // Recover from the panic below.
- _ = recover()
- // Restore cmdutil behavior
- cmdutil.DefaultBehaviorOnFatal()
- }()
- cmdutil.BehaviorOnFatal(func(e string, code int) { saw_fatal = true; panic(e) })
- cmd.SetArgs(test.args)
- cmd.Execute()
- }()
- if test.expectFatal {
- if !saw_fatal {
- t.Fatalf("%s: unexpected non-error", test.description)
- }
- }
- if test.expectTaint {
- if !tainted {
- t.Fatalf("%s: node not tainted", test.description)
- }
- }
- if !test.expectTaint {
- if tainted {
- t.Fatalf("%s: unexpected taint", test.description)
- }
- }
- }
- }
|