Update SmallRye Config to 2.11.0

[radcortez]

https://github.com/smallrye/smallrye-config/releases/tag/2.11.0

• Fixes Config mapping to a Map with a List value type fails #24944

[radcortez]

@gsmet I'll leave this to your consideration if you want to backport it to 2.11. There are a lot of code changes on SR Config, but they should not impact Quarkus.

[gsmet]

OK, I will be a bit conservative here as SR Config is really a critical component.

[radcortez]

Sure.

[quarkus-bot]

Failing Jobs - Building 280b478

Status	Name	Step	Failures	Logs	Raw logs
✔️	JVM Tests - JDK 11
✖	JVM Tests - JDK 11 Windows	Build	Failures	Logs	Raw logs
✔️	JVM Tests - JDK 17
✔️	JVM Tests - JDK 18

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/grpc/deployment 
! Skipped: extensions/opentelemetry/opentelemetry-exporter-jaeger/deployment extensions/opentelemetry/opentelemetry-exporter-otlp/deployment extensions/opentelemetry/opentelemetry/deployment and 19 more

📦 extensions/grpc/deployment

✖ io.quarkus.grpc.server.devmode.GrpcDevModeTest.testEchoStreamReload - More details - Source on GitHub

java.util.concurrent.RejectedExecutionException
	at org.jboss.threads.RejectingExecutor.execute(RejectingExecutor.java:38)
	at org.jboss.threads.EnhancedQueueExecutor.rejectShutdown(EnhancedQueueExecutor.java:2076)

[radcortez]

/cc @edeandrea

[edeandrea]

Thanks @radcortez ! Does this also pull in smallrye/smallrye-config#767?

[radcortez]

Thanks @radcortez ! Does this also pull in smallrye/smallrye-config#767?

Yes.

[edeandrea]

Thank you for keeping me in the loop!

[edeandrea]

Hey @radcortez this doesn't seem to work?

When I try this out using

%knative.quarkus.config.profile.parent=prod
%knative.quarkus.kubernetes.deployment-target=knative
%knative-openshift.quarkus.config.profile.parent=knative
%knative-openshift.quarkus.container-image.builder=openshift
%knative-openshift-17.quarkus.config.profile.parent=knative-openshift
# Some other %knative-openshift-17 properties below

and then I run

 ./mvnw clean package -Dquarkus.kubernetes.deploy=true -Dquarkus.profile=knative-openshift-17 -Dquarkus.kubernetes-client.trust-certs=true -DskipTests -Dquarkus.container-image.group=eric-deandrea-dev -Dquarkus.container-image.registry=

I would expect it to pick up quarkus.kubernetes.deployment-target=knative, but it doesn't. It does, however, pick up quarkus.container-image.builder=openshift.

Does the parent/child hierarchy only traverse 1 level?

If you'd like me to open up a separate issue let me know.

[radcortez]

No, it is probably related to the relocations of the smallrye to the quarkus namespace. If you use smallrye.config instead it works as expected.

I'll have a look.

[edeandrea]

Hi @radcortez I just wanted to circle back and see if you've had a chance to take a look at this?

[radcortez]

I believe this should have taken care of it: #28960

[edeandrea]

Thanks @radcortez I'll give it a go!

[edeandrea]

Thanks @radcortez - I've verified all is good! See quarkusio/quarkus-super-heroes#232

[radcortez]

Great!