mthoemme added a comment - 2 days ago
It'd be great if we could review the docs about configuring the autoscaler to check if the difference between containerConcurrency and the /target annotation are documented properly. 

In a nutshell:

• `.../target:` Should be the way to configure the autoscaler, period.
• `containerConcurrency` is a setting that should only be used, if your application needs to have an enforced constraint of concurrency (due to resource starvation or programming models... yaddayadda)

I think `containerConcurrency` might be too prominently advertised today (it's in the spec for instance) and we should have a look at balancing that in the docs.
Permalink
mgencur added a comment - 2 days ago - edited
Another piece of documentation upstream is https://www.knative.dev/docs/serving/samples/autoscale-go/index.html

There's no mention of "containerConcurrency" and all examples use the `/target` annotation. 

I think it's especially important include a note about the original "containerConcurrency" and when NOT to use it.