// Copyright © 2019 The Knative 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 service import ( "errors" "fmt" "reflect" "testing" "github.com/knative/client/pkg/kn/commands" servinglib "github.com/knative/client/pkg/serving" "github.com/knative/serving/pkg/apis/serving/v1alpha1" corev1 "k8s.io/api/core/v1" "k8s.io/apimachinery/pkg/api/resource" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/runtime" client_testing "k8s.io/client-go/testing" ) func fakeServiceUpdate(original *v1alpha1.Service, args []string) ( action client_testing.Action, updated *v1alpha1.Service, output string, err error) { knParams := &commands.KnParams{} cmd, fakeServing, buf := commands.CreateTestKnCommand(NewServiceCommand(knParams), knParams) fakeServing.AddReactor("update", "*", func(a client_testing.Action) (bool, runtime.Object, error) { updateAction, ok := a.(client_testing.UpdateAction) action = updateAction if !ok { return true, nil, fmt.Errorf("wrong kind of action %v", action) } updated, ok = updateAction.GetObject().(*v1alpha1.Service) if !ok { return true, nil, errors.New("was passed the wrong object") } return true, updated, nil }) fakeServing.AddReactor("get", "*", func(a client_testing.Action) (bool, runtime.Object, error) { return true, original, nil }) cmd.SetArgs(args) err = cmd.Execute() if err != nil { return } output = buf.String() return } func TestServiceUpdateImage(t *testing.T) { orig := newEmptyService() template, err := servinglib.GetRevisionTemplate(orig) if err != nil { t.Fatal(err) } servinglib.UpdateImage(template, "gcr.io/foo/bar:baz") action, updated, _, err := fakeServiceUpdate(orig, []string{ "service", "update", "foo", "--image", "gcr.io/foo/quux:xyzzy"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } template, err = servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } else if template.Spec.DeprecatedContainer.Image != "gcr.io/foo/quux:xyzzy" { t.Fatalf("wrong image set: %v", template.Spec.DeprecatedContainer.Image) } } func TestServiceUpdateMaxMinScale(t *testing.T) { original := newEmptyService() action, updated, _, err := fakeServiceUpdate(original, []string{ "service", "update", "foo", "--min-scale", "1", "--max-scale", "5", "--concurrency-target", "10", "--concurrency-limit", "100"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } template, err := servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } actualAnnos := template.Annotations expectedAnnos := []string{ "autoscaling.knative.dev/minScale", "1", "autoscaling.knative.dev/maxScale", "5", "autoscaling.knative.dev/target", "10", } for i := 0; i < len(expectedAnnos); i += 2 { anno := expectedAnnos[i] if actualAnnos[anno] != expectedAnnos[i+1] { t.Fatalf("Unexpected annotation value for %s : %s (actual) != %s (expected)", anno, actualAnnos[anno], expectedAnnos[i+1]) } } if template.Spec.ContainerConcurrency != 100 { t.Fatalf("container concurrency not set to given value 1000") } } func TestServiceUpdateEnv(t *testing.T) { orig := &v1alpha1.Service{ TypeMeta: metav1.TypeMeta{ Kind: "Service", APIVersion: "knative.dev/v1alpha1", }, ObjectMeta: metav1.ObjectMeta{ Name: "foo", Namespace: "default", }, Spec: v1alpha1.ServiceSpec{ DeprecatedRunLatest: &v1alpha1.RunLatestType{ Configuration: v1alpha1.ConfigurationSpec{ DeprecatedRevisionTemplate: &v1alpha1.RevisionTemplateSpec{ Spec: v1alpha1.RevisionSpec{ DeprecatedContainer: &corev1.Container{}, }, }, }, }, }, } template, err := servinglib.GetRevisionTemplate(orig) if err != nil { t.Fatal(err) } servinglib.UpdateImage(template, "gcr.io/foo/bar:baz") action, updated, _, err := fakeServiceUpdate(orig, []string{ "service", "update", "foo", "-e", "TARGET=Awesome"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } expectedEnvVar := corev1.EnvVar{ Name: "TARGET", Value: "Awesome", } template, err = servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } else if template.Spec.DeprecatedContainer.Image != "gcr.io/foo/bar:baz" { t.Fatalf("wrong image set: %v", template.Spec.DeprecatedContainer.Image) } else if template.Spec.DeprecatedContainer.Env[0] != expectedEnvVar { t.Fatalf("wrong env set: %v", template.Spec.DeprecatedContainer.Env) } } func TestServiceUpdateRequestsLimitsCPU(t *testing.T) { service := createMockServiceWithResources(t, "250", "64Mi", "1000m", "1024Mi") action, updated, _, err := fakeServiceUpdate(service, []string{ "service", "update", "foo", "--requests-cpu", "500m", "--limits-cpu", "1000m"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } expectedRequestsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("500m"), corev1.ResourceMemory: resource.MustParse("64Mi"), } expectedLimitsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("1000m"), corev1.ResourceMemory: resource.MustParse("1024Mi"), } newTemplate, err := servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } else { if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Requests, expectedRequestsVars) { t.Fatalf("wrong requests vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Requests) } if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Limits, expectedLimitsVars) { t.Fatalf("wrong limits vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Limits) } } } func TestServiceUpdateRequestsLimitsMemory(t *testing.T) { service := createMockServiceWithResources(t, "100m", "64Mi", "1000m", "1024Mi") action, updated, _, err := fakeServiceUpdate(service, []string{ "service", "update", "foo", "--requests-memory", "128Mi", "--limits-memory", "2048Mi"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } expectedRequestsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("100m"), corev1.ResourceMemory: resource.MustParse("128Mi"), } expectedLimitsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("1000m"), corev1.ResourceMemory: resource.MustParse("2048Mi"), } newTemplate, err := servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } else { if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Requests, expectedRequestsVars) { t.Fatalf("wrong requests vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Requests) } if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Limits, expectedLimitsVars) { t.Fatalf("wrong limits vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Limits) } } } func TestServiceUpdateRequestsLimitsCPU_and_Memory(t *testing.T) { service := createMockServiceWithResources(t, "250m", "64Mi", "1000m", "1024Mi") action, updated, _, err := fakeServiceUpdate(service, []string{ "service", "update", "foo", "--requests-cpu", "500m", "--limits-cpu", "2000m", "--requests-memory", "128Mi", "--limits-memory", "2048Mi"}) if err != nil { t.Fatal(err) } else if !action.Matches("update", "services") { t.Fatalf("Bad action %v", action) } expectedRequestsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("500m"), corev1.ResourceMemory: resource.MustParse("128Mi"), } expectedLimitsVars := corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse("2000m"), corev1.ResourceMemory: resource.MustParse("2048Mi"), } newTemplate, err := servinglib.GetRevisionTemplate(updated) if err != nil { t.Fatal(err) } else { if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Requests, expectedRequestsVars) { t.Fatalf("wrong requests vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Requests) } if !reflect.DeepEqual( newTemplate.Spec.DeprecatedContainer.Resources.Limits, expectedLimitsVars) { t.Fatalf("wrong limits vars %v", newTemplate.Spec.DeprecatedContainer.Resources.Limits) } } } func newEmptyService() *v1alpha1.Service { return &v1alpha1.Service{ TypeMeta: metav1.TypeMeta{ Kind: "Service", APIVersion: "knative.dev/v1alpha1", }, ObjectMeta: metav1.ObjectMeta{ Name: "foo", Namespace: "default", }, Spec: v1alpha1.ServiceSpec{ DeprecatedRunLatest: &v1alpha1.RunLatestType{ Configuration: v1alpha1.ConfigurationSpec{ DeprecatedRevisionTemplate: &v1alpha1.RevisionTemplateSpec{ Spec: v1alpha1.RevisionSpec{ DeprecatedContainer: &corev1.Container{}, }, }, }, }, }, } } func createMockServiceWithResources(t *testing.T, requestCPU, requestMemory, limitsCPU, limitsMemory string) *v1alpha1.Service { service := &v1alpha1.Service{ TypeMeta: metav1.TypeMeta{ Kind: "Service", APIVersion: "knative.dev/v1alpha1", }, ObjectMeta: metav1.ObjectMeta{ Name: "foo", Namespace: "default", }, Spec: v1alpha1.ServiceSpec{ DeprecatedRunLatest: &v1alpha1.RunLatestType{ Configuration: v1alpha1.ConfigurationSpec{ DeprecatedRevisionTemplate: &v1alpha1.RevisionTemplateSpec{ Spec: v1alpha1.RevisionSpec{ DeprecatedContainer: &corev1.Container{}, }, }, }, }, }, } template, err := servinglib.GetRevisionTemplate(service) if err != nil { t.Fatal(err) } template.Spec.DeprecatedContainer.Resources = corev1.ResourceRequirements{ Requests: corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse(requestCPU), corev1.ResourceMemory: resource.MustParse(requestMemory), }, Limits: corev1.ResourceList{ corev1.ResourceCPU: resource.MustParse(limitsCPU), corev1.ResourceMemory: resource.MustParse(limitsMemory), }, } return service }