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

Update SmallRye Config to 3.10.0 #42227

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

radcortez
Copy link
Member

@radcortez radcortez commented Jul 30, 2024

SmallRye Config Release notes:

3.10.0

  • #1242 Bump version.curator from 5.7.0 to 5.7.1
  • #1240 Bump kotlin.version from 2.0.20 to 2.0.21
  • #1239 Add @ConfigMapping beanStyleGetter to enable / disable bean style getter names matching with configuration names
  • #1237 Bump org.ow2.asm:asm from 9.7 to 9.7.1
  • #1236 Rename ConfigClassWithPrefix to ConfigClass and use it in SmallRyeConfigBuilder
  • #1235 Bump version.smallrye.testing from 2.3.0 to 2.3.1
  • #1234 Bump io.smallrye.common:smallrye-common-bom from 2.4.0 to 2.7.0
  • #1231 Slight optimization when looking up System properties
  • #1229 Bump io.fabric8:docker-maven-plugin from 0.45.0 to 0.45.1
  • #1226 Move local classes to inner to reduce the number of classes in the main package
  • #1224 Remove constructor arguments from examples
  • #1223 Fix docs of interceptor service registration
  • #1222 Bump io.smallrye:smallrye-parent from 45 to 46
  • #1220 Support a fixed list of Map keys statically @WithKeys
  • #1218 Bump org.yaml:snakeyaml from 2.2 to 2.3
  • #1217 Cache profile prefixes
  • #1216 Avoid expensive exception and log when getValues fails lookup for indexed properties and fallbacks to comma
  • #1215 Check if profile file resources are in the location ClassLoader
  • #1214 Internal cleanup of AbstractLocationConfigSourceLoader
  • #1213 Reduce allocations of iterateNames
  • #1212 Improve mappings documentation
  • #1211 Avoid using string concatenation to forge impl name
  • #1210 Bump kotlin.version from 2.0.0 to 2.0.20
  • #1209 ConfigValue name consistent with PropertiesConfigSource
  • #1204 Search for indexed property names before flattened comma separated value name when loading Collections for CDI injection
  • #1203 Remove the generation of a comma separated value name for Collections in the YamlConfigSource
  • #1202 Search for indexed property names before flattened comma separated value name when loading Collections
  • #1201 Drop support for full YAML content in parent property names
  • #1200 Bump io.fabric8:docker-maven-plugin from 0.44.0 to 0.45.0
  • #1198 Update sample ordinal in custom.md
  • #1195 Bump zipp from 3.15.0 to 3.19.1 in /documentation

Quarkus:

Breaking Changes:

  • Search for indexed property names before flattened comma separated value name when loading Collections
  • Remove the generation of a comma separated value name for Collections in the YamlConfigSource

@quarkus-bot quarkus-bot bot added area/config area/dependencies Pull requests that update a dependency file area/smallrye labels Jul 30, 2024
@quarkusio quarkusio deleted a comment from quarkus-bot bot Sep 17, 2024
@quarkus-bot quarkus-bot bot added area/arc Issue related to ARC (dependency injection) area/core area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/testing labels Oct 7, 2024
@radcortez radcortez marked this pull request as ready for review October 15, 2024 23:19

This comment has been minimized.

Copy link

github-actions bot commented Oct 15, 2024

🎊 PR Preview de27583 has been successfully built and deployed to https://quarkus-pr-main-42227-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

This comment has been minimized.

@radcortez radcortez requested a review from geoand October 16, 2024 07:43
@geoand geoand removed area/core area/arc Issue related to ARC (dependency injection) area/smallrye area/testing area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins labels Oct 16, 2024
@quarkus-bot quarkus-bot bot added area/arc Issue related to ARC (dependency injection) area/core area/testing labels Oct 16, 2024
@gsmet
Copy link
Member

gsmet commented Oct 16, 2024

Not sure what's going on but the main CI was skipped.

This comment has been minimized.

@radcortez
Copy link
Member Author

Hum... strange.

This comment has been minimized.

@radcortez
Copy link
Member Author

#43345 fixes the failure of io.quarkus.rest.client.reactive.ConfigurationTest.

Copy link

quarkus-bot bot commented Oct 21, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 848bfb4.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

Copy link

quarkus-bot bot commented Oct 21, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 848bfb4.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 17 Build Failures Logs Raw logs 🔍
JVM Tests - JDK 21 Build Failures Logs Raw logs 🔍
JVM Tests - JDK 17 Windows Build Failures Logs Raw logs 🔍

Full information is available in the Build summary check run.
You can consult the Develocity build scans.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/resteasy-reactive/rest-client/deployment 
! Skipped: extensions/keycloak-admin-rest-client/deployment extensions/micrometer-registry-prometheus/deployment extensions/micrometer/deployment and 59 more

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

io.quarkus.rest.client.reactive.ConfigurationTest. - History - More details - Source on GitHub

java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:736)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
	at io.quarkus.runtime.Application.start(Application.java:101)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:305)

⚙️ JVM Tests - JDK 21 #

- Failing: extensions/resteasy-reactive/rest-client/deployment 
! Skipped: extensions/keycloak-admin-rest-client/deployment extensions/micrometer-registry-prometheus/deployment extensions/micrometer/deployment and 59 more

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

io.quarkus.rest.client.reactive.ConfigurationTest. - History - More details - Source on GitHub

java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:736)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
	at io.quarkus.runtime.Application.start(Application.java:101)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:305)

⚙️ JVM Tests - JDK 17 Windows #

- Failing: extensions/resteasy-reactive/rest-client/deployment 
! Skipped: extensions/keycloak-admin-rest-client/deployment extensions/micrometer-registry-prometheus/deployment extensions/micrometer/deployment and 59 more

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

io.quarkus.rest.client.reactive.ConfigurationTest. - History - More details - Source on GitHub

java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:736)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
	at io.quarkus.runtime.Application.start(Application.java:101)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:305)

Flaky tests - Develocity

⚙️ JVM Tests - JDK 17

📦 extensions/smallrye-reactive-messaging/deployment

io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector - History

  • Expecting actual: ["-6","-7","-9","-10","-11","-12","-13","-14"] to start with: ["-6", "-7", "-8", "-9"] - java.lang.AssertionError
java.lang.AssertionError: 

Expecting actual:
  ["-6","-7","-9","-10","-11","-12","-13","-14"]
to start with:
  ["-6", "-7", "-8", "-9"]

	at io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector(ConnectorChangeTest.java:41)

@gsmet gsmet merged commit 848bfb4 into quarkusio:main Oct 22, 2024
52 of 55 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.17 - main milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants