client icon indicating copy to clipboard operation
client copied to clipboard

Add `--scale-activation` flag to match new Serving option

Open dsimansk opened this issue 3 years ago • 4 comments

Feature request

Consider adding a new Ksvc flag to specify activation scale value. That represents number of replicas to be scaled from zero when service is activated.

Ref: https://github.com/knative/serving/pull/13161 (note it's an update PR to introduce new & current name of option) https://github.com/knative/serving/pull/13136 (first PR with former name)

UI Example

kn service create hello --image .../myapp --scale-activation 2

/kind good-first-issue

/cc @dprotaso @psschwei

dsimansk avatar Jul 29 '22 10:07 dsimansk

+1, maybe we should think about the naming, to better reflect the annotation. What's about --scale-min-active ?

rhuss avatar Jul 29 '22 11:07 rhuss

+1, maybe we should think about the naming, to better reflect the annotation. What's about --scale-min-active ?

It was renamed in Serving's PR #13161 to activation-scale. I was a bit confused too, but that's the current naming on main https://github.com/knative/serving/blob/main/pkg/apis/autoscaling/register.go#L220-L225.

dsimansk avatar Jul 29 '22 13:07 dsimansk

Yeah, we went back and forth on the name just a bit :smile: There's a fairly long slack thread where we went over the various iterations.

psschwei avatar Jul 29 '22 13:07 psschwei

Yeah, we went back and forth on the name just a bit 😄 There's a fairly long slack thread where we went over the various iterations.

ah, what a pity that I missed that discussion. Love reasoning about naming ;-P

rhuss avatar Aug 16 '22 12:08 rhuss

/assign

vyasgun avatar Aug 23 '22 07:08 vyasgun