Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.7.3 backports 2 #24043

Merged
merged 11 commits into from
Mar 2, 2022
Merged

2.7.3 backports 2 #24043

merged 11 commits into from
Mar 2, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Mar 1, 2022

Please don't merge, I will merge it myself.

michalszynkiewicz and others added 11 commits March 1, 2022 21:03
This PR the following issues:
- The generated `config-autoscaler` configmap does not set the namespace which should always be `knative-serving` (see for example https://knative.dev/docs/serving/autoscaling/scale-to-zero/#enable-scale-to-zero).
- Some properties do add the annotations at Knative service metadata level, and it should add them at Knative service spec template metadata level.
- The property `quarkus.knative.global-auto-scaling.containerConcurrency` (hard limit option) wrongly uses the `config-autoscaler` configmap and it should use `config-defaults` configmap (see https://knative.dev/docs/serving/autoscaling/concurrency/#hard-limit)

Moreover, add the autoscaling Knative properties to the Knative documentation.

Fix quarkusio#23786

(cherry picked from commit 3f797dc)
…jo execution in case Maven extensions are disabled

(cherry picked from commit 3c454d5)
…evel

I only exposed the constraint one and not the custom violation one as I
think it is bad practice to globally change the custom violation one.

Fix quarkusio#23838

(cherry picked from commit 780d683)
@gsmet gsmet added the area/infra internal and infrastructure related issues label Mar 1, 2022
@gsmet gsmet self-assigned this Mar 1, 2022
@gsmet gsmet merged commit 99d643d into quarkusio:2.7 Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants