Skip to content

Conversation

cdsap
Copy link
Contributor

@cdsap cdsap commented Jun 25, 2025

We observed configuration cache invalidations when changing arbitrary system properties via the command line.

The root cause is the usage of System.getProperties() in the extractQuarkusTestSystemProperties method, which captures all system properties and breaks cacheability.

This PR replaces the general System.getProperties() call with Gradle’s systemPropertiesPrefixedBy provider. This returns a name-to-value map of system properties filtered by the given prefix.

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle labels Jun 25, 2025
@geoand geoand requested a review from gsmet June 26, 2025 05:38
Copy link

quarkus-bot bot commented Jun 26, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 9ccf42e.

✅ 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.

You can consult the Develocity build scans.

@gsmet gsmet merged commit 3d94616 into quarkusio:main Jun 26, 2025
23 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.25 - main milestone Jun 26, 2025
@gsmet
Copy link
Member

gsmet commented Jun 26, 2025

Thanks!

@gsmet gsmet modified the milestones: 3.25 - main, 3.24.2 Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants