Move --autoscale-window to --scale-window

CHANGELOG.adoc

@@ -11,6 +11,18 @@
 |
 | https://github.com/knative/client/pull/[#]
 ////
+
+## v1.0.0
+[cols="1,10,3", options="header", width="100%"]
+|===
+| | Description | PR
+
+| ✨
+| Deprecate `--autoscale-window` in favor of `--scale-window`
+| https://github.com/knative/client/pull/1454[#1454]
+
+|===
+
 ## v0.26.0 (2021-09-22)
 [cols="1,10,3", options="header", width="100%"]
 |===

pkg/kn/commands/service/configuration_edit_flags.go

@@ -44,7 +44,7 @@ type ConfigurationEditFlags struct {
 	ConcurrencyTarget      int
 	ConcurrencyLimit       int
 	ConcurrencyUtilization int
-	AutoscaleWindow        string
+	ScaleWindow            string
 	Labels                 []string
 	LabelsService          []string
 	LabelsRevision         []string
@@ -100,9 +100,13 @@ func (p *ConfigurationEditFlags) addSharedFlags(command *cobra.Command) {
 	command.Flags().IntVar(&p.MaxScale, "scale-max", 0, "Maximum number of replicas.")
 	p.markFlagMakesRevision("scale-max")
 
-	command.Flags().StringVar(&p.AutoscaleWindow, "autoscale-window", "", "Duration to look back for making auto-scaling decisions. The service is scaled to zero if no request was received in during that time. (eg: 10s)")
+	// DEPRECATED since 1.0
+	command.Flags().StringVar(&p.ScaleWindow, "autoscale-window", "", "Deprecated option, please use --scale-window")
 	p.markFlagMakesRevision("autoscale-window")
 
+	command.Flags().StringVar(&p.ScaleWindow, "scale-window", "", "Duration to look back for making auto-scaling decisions. The service is scaled to zero if no request was received in during that time. (eg: 10s)")
+	p.markFlagMakesRevision("scale-window")
+
 	knflags.AddBothBoolFlagsUnhidden(command.Flags(), &p.ClusterLocal, "cluster-local", "", false,
 		"Specify that the service be private. (--no-cluster-local will make the service publicly available)")
 
@@ -315,7 +319,7 @@ func (p *ConfigurationEditFlags) Apply(
 	}
 
 	if cmd.Flags().Changed("autoscale-window") {
-		err = servinglib.UpdateAutoscaleWindow(template, p.AutoscaleWindow)
+		err = servinglib.UpdateScaleWindow(template, p.ScaleWindow)
 		if err != nil {
 			return err
 		}

pkg/kn/commands/service/create_mock_test.go

@@ -539,7 +539,7 @@ func TestServiceCreateWithAnnotations(t *testing.T) {
 	output, err := executeServiceCommand(client, "create", "foo", "--image", "gcr.io/foo/bar:baz",
 		"--annotation", "foo=bar",
 		"--annotation", autoscaling.InitialScaleAnnotationKey+"=1",
-		"--no-wait", "--revision-name=")
+		"--no-wait")
 	assert.NilError(t, err)
 	assert.Assert(t, util.ContainsAll(output, "created", "foo", "default"))
 
@@ -570,7 +570,7 @@ func TestServiceCreateWithRevisionAndServiceAnnotations(t *testing.T) {
 	output, err := executeServiceCommand(client, "create", "foo", "--image", "gcr.io/foo/bar:baz",
 		"--annotation-service", "foo=bar",
 		"--annotation-revision", autoscaling.InitialScaleAnnotationKey+"=1",
-		"--no-wait", "--revision-name=")
+		"--no-wait")
 	assert.NilError(t, err)
 	assert.Assert(t, util.ContainsAll(output, "created", "foo", "default"))
 
@@ -596,7 +596,7 @@ func TestServiceCreateWithRevisionAnnotations(t *testing.T) {
 
 	output, err := executeServiceCommand(client, "create", "foo", "--image", "gcr.io/foo/bar:baz",
 		"--annotation-revision", autoscaling.InitialScaleAnnotationKey+"=1",
-		"--no-wait", "--revision-name=")
+		"--no-wait")
 	assert.NilError(t, err)
 	assert.Assert(t, util.ContainsAll(output, "created", "foo", "default"))
 
@@ -625,21 +625,21 @@ func TestServiceCreateWithServiceAnnotations(t *testing.T) {
 
 	output, err := executeServiceCommand(client, "create", "foo", "--image", "gcr.io/foo/bar:baz",
 		"--annotation-service", "foo=bar",
-		"--no-wait", "--revision-name=")
+		"--no-wait")
 	assert.NilError(t, err)
 	assert.Assert(t, util.ContainsAll(output, "created", "foo", "default"))
 
 	r.Validate()
 }
 
-func TestServiceCreateWithAutoScaleServiceAnnotationsError(t *testing.T) {
+func TestServiceCreateWithScaleServiceAnnotationsError(t *testing.T) {
 	client := knclient.NewMockKnServiceClient(t)
 
 	r := client.Recorder()
 
 	output, err := executeServiceCommand(client, "create", "foo", "--image", "gcr.io/foo/bar:baz",
 		"--annotation-service", autoscaling.InitialScaleAnnotationKey+"=1",
-		"--no-wait", "--revision-name=")
+		"--no-wait")
 	assert.Assert(t, err != nil)
 	assert.Assert(t, util.ContainsAll(output, "service can not have auto-scaling related annotation", "autoscaling.knative.dev/initialScale"))
 

pkg/serving/config_changes.go

@@ -64,11 +64,11 @@ func UpdateMaxScale(template *servingv1.RevisionTemplateSpec, max int) error {
 	return UpdateRevisionTemplateAnnotation(template, autoscaling.MaxScaleAnnotationKey, strconv.Itoa(max))
 }
 
-// UpdateAutoscaleWindow updates the autoscale window annotation
-func UpdateAutoscaleWindow(template *servingv1.RevisionTemplateSpec, window string) error {
+// UpdateScaleWindow updates the autoscale window annotation
+func UpdateScaleWindow(template *servingv1.RevisionTemplateSpec, window string) error {
 	_, err := time.ParseDuration(window)
 	if err != nil {
-		return fmt.Errorf("invalid duration for 'autoscale-window': %w", err)
+		return fmt.Errorf("invalid duration for 'scale-window': %w", err)
 	}
 	return UpdateRevisionTemplateAnnotation(template, autoscaling.WindowAnnotationKey, window)
 }

pkg/serving/config_changes_test.go

@@ -31,7 +31,7 @@ import (
 	servingv1 "knative.dev/serving/pkg/apis/serving/v1"
 )
 
-func TestUpdateAutoscalingAnnotations(t *testing.T) {
+func TestUpdateScalingAnnotations(t *testing.T) {
 	template := &servingv1.RevisionTemplateSpec{}
 	updateConcurrencyConfiguration(template, 10, 100, 1000, 1000, 50)
 	annos := template.Annotations
@@ -49,7 +49,7 @@ func TestUpdateAutoscalingAnnotations(t *testing.T) {
 	}
 }
 
-func TestUpdateInvalidAutoscalingAnnotations(t *testing.T) {
+func TestUpdateInvalidScalingAnnotations(t *testing.T) {
 	template := &servingv1.RevisionTemplateSpec{}
 	updateConcurrencyConfiguration(template, 10, 100, 1000, 1000, 50)
 	// Update with invalid concurrency options
@@ -169,15 +169,15 @@ func TestUpdateMaxScale(t *testing.T) {
 	assert.ErrorContains(t, err, "maxScale")
 }
 
-func TestAutoscaleWindow(t *testing.T) {
+func TestScaleWindow(t *testing.T) {
 	template, _ := getRevisionTemplate()
-	err := UpdateAutoscaleWindow(template, "10s")
+	err := UpdateScaleWindow(template, "10s")
 	assert.NilError(t, err)
 	// Verify update is successful or not
 	checkAnnotationValue(t, template, autoscaling.WindowAnnotationKey, "10s")
 	// Update with invalid value
-	err = UpdateAutoscaleWindow(template, "blub")
-	assert.Check(t, util.ContainsAll(err.Error(), "invalid duration", "autoscale-window"))
+	err = UpdateScaleWindow(template, "blub")
+	assert.Check(t, util.ContainsAll(err.Error(), "invalid duration", "scale-window"))
 }
 
 func TestUpdateConcurrencyTarget(t *testing.T) {

test/e2e/service_options_test.go

@@ -109,9 +109,9 @@ func TestServiceOptions(t *testing.T) {
 	test.ServiceDelete(r, "svc3a")
 
 	t.Log("create, update and validate service with autoscale window option")
-	serviceCreateWithOptions(r, "svc4", "--autoscale-window", "1m")
+	serviceCreateWithOptions(r, "svc4", "--scale-window", "1m")
 	validateAutoscaleWindow(r, "svc4", "1m")
-	test.ServiceUpdate(r, "svc4", "--autoscale-window", "15s")
+	test.ServiceUpdate(r, "svc4", "--scale-window", "15s")
 	validateAutoscaleWindow(r, "svc4", "15s")
 	test.ServiceDelete(r, "svc4")