Update SmallRye Config to 2.8.0 #22423

radcortez · 2021-12-20T22:58:36Z

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

Fixes Optional mapping error with ConfigMapping using Maps with enum keys, Nested Groups and Optional #22334

gsmet · 2021-12-20T23:15:28Z

Is it something we want for 2.6.1.Final or we wait for 2.7?

radcortez · 2021-12-20T23:18:05Z

I believe we can wait until 2.7. If required users can update manually for now. The version is compatible, I've just bumped the minor due to a new public API (which is not used in Quarkus yet).

quarkus-bot · 2021-12-21T02:32:02Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `808e91f`

Status	Name	Step	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11 Windows	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 17	`Build`	Failures	Logs	Raw logs
✖	Native Tests - Misc3	`Build`	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: integration-tests/smallrye-config

📦 integration-tests/smallrye-config

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationYamlProfile line 65 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.locations line 26 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationPropertiesProfile line 50 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.dotenv line 40 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.properties line 55 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.mpConfigProperties line 25 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

⚙️ JVM Tests - JDK 11 Windows #

- Failing: integration-tests/smallrye-config

📦 integration-tests/smallrye-config

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationYamlProfile line 65 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.locations line 26 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationPropertiesProfile line 50 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.dotenv line 40 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.properties line 55 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.mpConfigProperties line 25 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

⚙️ JVM Tests - JDK 17 #

- Failing: integration-tests/smallrye-config

📦 integration-tests/smallrye-config

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationYamlProfile line 65 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.locations line 26 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsTest.applicationPropertiesProfile line 50 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.dotenv line 40 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.properties line 55 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigTest.mpConfigProperties line 25 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

⚙️ Native Tests - Misc3 #

- Failing: integration-tests/smallrye-config

📦 integration-tests/smallrye-config

✖ io.quarkus.it.smallrye.config.ConfigLocationsIT.applicationYamlProfile - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsIT.locations - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.ConfigLocationsIT.applicationPropertiesProfile - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigIT.dotenv - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigIT.properties - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

✖ io.quarkus.it.smallrye.config.SmallRyeConfigIT.mpConfigProperties - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
JSON path value doesn't match.

radcortez · 2021-12-21T10:54:04Z

Actually, we need some additional code changes, so let's do this only for 2.7.

quarkus-bot · 2021-12-21T14:59:46Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `e2fdd9e`

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

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: extensions/resteasy-reactive/rest-client-reactive/deployment extensions/smallrye-reactive-messaging-kafka/deployment 
! Skipped: extensions/oidc-client-reactive-filter/deployment extensions/resteasy-reactive/rest-client-reactive-jackson/deployment extensions/resteasy-reactive/rest-client-reactive-kotlin-serialization/deployment and 12 more

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

✖ io.quarkus.rest.client.reactive.LegacyConfigurationTest.configurationShouldBeLoaded line 29 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

Expecting value to be true but was false

📦 extensions/smallrye-reactive-messaging-kafka/deployment

✖ io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingWorkingAppPropsTestCase.testContinuousTestingScenario3 - More details - Source on GitHub

java.lang.RuntimeException: 
java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkus.kafka.client.deployment.DevServicesKafkaProcessor#startKafkaDevService threw an exception: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/resteasy-reactive/rest-client-reactive/deployment 
! Skipped: extensions/oidc-client-reactive-filter/deployment extensions/resteasy-reactive/rest-client-reactive-jackson/deployment extensions/resteasy-reactive/rest-client-reactive-kotlin-serialization/deployment and 8 more

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

✖ io.quarkus.rest.client.reactive.LegacyConfigurationTest.configurationShouldBeLoaded line 29 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

Expecting value to be true but was false

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/resteasy-reactive/rest-client-reactive/deployment 
! Skipped: extensions/oidc-client-reactive-filter/deployment extensions/resteasy-reactive/rest-client-reactive-jackson/deployment extensions/resteasy-reactive/rest-client-reactive-kotlin-serialization/deployment and 8 more

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

✖ io.quarkus.rest.client.reactive.LegacyConfigurationTest.configurationShouldBeLoaded line 29 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

Expecting value to be true but was false

quarkus-bot · 2021-12-22T20:50:38Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `fe6d9d0`

Status	Name	Step	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11 Windows	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 17	`Build`	Failures	Logs	Raw logs
✖	Native Tests - Misc4	`Build`	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: extensions/resteasy-classic/rest-client-config/runtime 
! Skipped: extensions/keycloak-admin-client/deployment extensions/keycloak-admin-client/runtime extensions/keycloak-authorization/deployment and 96 more

📦 extensions/resteasy-classic/rest-client-config/runtime

✖ io.quarkus.restclient.config.RestClientFallbackConfigSourceInterceptorTest.testGetValue line 97 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "prefix/mp-rest/url"

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/resteasy-classic/rest-client-config/runtime 
! Skipped: extensions/keycloak-admin-client/deployment extensions/keycloak-admin-client/runtime extensions/keycloak-authorization/deployment and 96 more

📦 extensions/resteasy-classic/rest-client-config/runtime

✖ io.quarkus.restclient.config.RestClientFallbackConfigSourceInterceptorTest.testGetValue line 97 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "prefix/mp-rest/url"

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/resteasy-classic/rest-client-config/runtime 
! Skipped: extensions/keycloak-admin-client/deployment extensions/keycloak-admin-client/runtime extensions/keycloak-authorization/deployment and 96 more

📦 extensions/resteasy-classic/rest-client-config/runtime

✖ io.quarkus.restclient.config.RestClientFallbackConfigSourceInterceptorTest.testGetValue line 97 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "prefix/mp-rest/url"

⚙️ Native Tests - Misc4 #

- Failing: integration-tests/picocli-native

📦 integration-tests/picocli-native

✖ io.quarkus.it.picocli.PicocliIT.testParentCommand(LaunchResult) - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "testValue"

✖ io.quarkus.it.picocli.PicocliIT.testDefaultValueProvider(LaunchResult) - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "default:default-value"

✖ io.quarkus.it.picocli.PicocliIT.testMethodSubCommand(QuarkusMainLauncher) - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "Hello World!"

✖ io.quarkus.it.picocli.PicocliIT.testDynamicProxy(LaunchResult) - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "2007-12-03T10:15:30"

✖ io.quarkus.it.picocli.PicocliIT.testLogCapturing(QuarkusMainLauncher) - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "Hello World!"

Record config properties with ConfigValue This will improve the support of the fallback and relocation mechanisms

gastaldi

LGTM

quarkus-bot bot added the area/dependencies Pull requests that update a dependency file label Dec 20, 2021

radcortez force-pushed the srconfig-280 branch from 808e91f to 9dde355 Compare December 21, 2021 10:23

quarkus-bot bot added area/config area/core labels Dec 21, 2021

radcortez force-pushed the srconfig-280 branch from 9dde355 to e2fdd9e Compare December 21, 2021 10:51

radcortez force-pushed the srconfig-280 branch from e2fdd9e to fe6d9d0 Compare December 22, 2021 18:02

Update SmallRye Config to 2.8.1

4c2c376

Record config properties with ConfigValue This will improve the support of the fallback and relocation mechanisms

radcortez force-pushed the srconfig-280 branch from fe6d9d0 to 4c2c376 Compare December 23, 2021 00:33

gastaldi approved these changes Dec 23, 2021

View reviewed changes

gastaldi merged commit 98b1459 into quarkusio:main Dec 23, 2021

quarkus-bot bot added this to the 2.7 - main milestone Dec 23, 2021

radcortez mentioned this pull request Jan 3, 2022

Record config properties with ConfigValue #22283

Closed

radcortez deleted the srconfig-280 branch January 3, 2022 11:14

radcortez mentioned this pull request Feb 15, 2022

NoSuchMethodError after Quarkus 2.7.1 uplift smallrye/smallrye-reactive-messaging#1609

Closed

Update SmallRye Config to 2.8.0 #22423

Update SmallRye Config to 2.8.0 #22423

Conversation

radcortez commented Dec 20, 2021 • edited Loading

gsmet commented Dec 20, 2021

radcortez commented Dec 20, 2021

quarkus-bot bot commented Dec 21, 2021 • edited Loading

Failing Jobs - Building 808e91f

Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/smallrye-config

⚙️ JVM Tests - JDK 11 Windows #

📦 integration-tests/smallrye-config

⚙️ JVM Tests - JDK 17 #

📦 integration-tests/smallrye-config

⚙️ Native Tests - Misc3 #

📦 integration-tests/smallrye-config

radcortez commented Dec 21, 2021

quarkus-bot bot commented Dec 21, 2021 • edited Loading

Failing Jobs - Building e2fdd9e

Failures

⚙️ JVM Tests - JDK 11 #

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

📦 extensions/smallrye-reactive-messaging-kafka/deployment

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

⚙️ JVM Tests - JDK 17 #

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

quarkus-bot bot commented Dec 22, 2021 • edited Loading

Failing Jobs - Building fe6d9d0

Failures

⚙️ JVM Tests - JDK 11 #

📦 extensions/resteasy-classic/rest-client-config/runtime

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/resteasy-classic/rest-client-config/runtime

⚙️ JVM Tests - JDK 17 #

📦 extensions/resteasy-classic/rest-client-config/runtime

⚙️ Native Tests - Misc4 #

📦 integration-tests/picocli-native

gastaldi left a comment

Choose a reason for hiding this comment

radcortez commented Dec 20, 2021 •

edited

Loading

quarkus-bot bot commented Dec 21, 2021 •

edited

Loading

Failing Jobs - Building `808e91f`

quarkus-bot bot commented Dec 21, 2021 •

edited

Loading

Failing Jobs - Building `e2fdd9e`

quarkus-bot bot commented Dec 22, 2021 •

edited

Loading

Failing Jobs - Building `fe6d9d0`