Skip to content

Conversation

manofthepeace
Copy link
Contributor

Fixes #50076

@quarkus-bot quarkus-bot bot added area/amazon-lambda area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/funqy area/kubernetes area/maven area/platform Issues related to definition and interaction with Quarkus Platform labels Sep 16, 2025

This comment has been minimized.

Copy link

github-actions bot commented Sep 16, 2025

🙈 The PR is closed and the preview is expired.

This comment has been minimized.

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

I think there's at least this one missing:

<version.surefire.plugin>3.5.3</version.surefire.plugin>
.

I know about it because we got some Dependabot errors due to the misalignment.

Maybe we should have a script that handles this? It has been quite a pain.

@manofthepeace
Copy link
Contributor Author

Done. Script may be a little hard as there are some cases like;

independent-projects/tools/devtools-testing/src/main/java/io/quarkus/devtools/testing/codestarts/QuarkusCodestartTesting.java
data.put(MAVEN_SUREFIRE_PLUGIN_VERSION.key(), "3.0.0-MOCK");

and this in a view places;
<surefire-plugin.version>3.0.0-MOCK</surefire-plugin.version>

The one I had missed was 3.5.3 instead of 3.5.2. I wonder if that was dependabot? Maybe it should not handle surefire if it causes mismatch?

I did another manual look and I believe it is fine now.

@manofthepeace manofthepeace requested a review from gsmet September 17, 2025 14:22
Copy link

quarkus-bot bot commented Sep 17, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 7350a1e.

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

@gastaldi gastaldi added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 17, 2025
Copy link

quarkus-bot bot commented Sep 17, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 7350a1e.

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


Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 test-framework/jacoco/runtime

io.quarkus.jacoco.runtime.DataFileWatchTest.waitForPreexistingDataFileThatNeverChanges - History

  • Expecting value to be false but was true - org.opentest4j.AssertionFailedError
org.opentest4j.AssertionFailedError: 

Expecting value to be false but was true
	at io.quarkus.jacoco.runtime.DataFileWatchTest.waitForPreexistingDataFileThatNeverChanges(DataFileWatchTest.java:241)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)

⚙️ Gradle Tests - JDK 17 Windows

📦 integration-tests/gradle

io.quarkus.gradle.devmode.IncludedKotlinBuildDevModeTest.main - History

  • Condition with Lambda expression in io.quarkus.test.devmode.util.DevModeClient was not fulfilled within 1 minutes 30 seconds. - org.awaitility.core.ConditionTimeoutException
org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in io.quarkus.test.devmode.util.DevModeClient was not fulfilled within 1 minutes  30 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1160)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1129)
	at io.quarkus.test.devmode.util.DevModeClient.getHttpResponse(DevModeClient.java:164)
	at io.quarkus.gradle.devmode.QuarkusDevGradleTestBase.getHttpResponse(QuarkusDevGradleTestBase.java:165)

@gastaldi gastaldi merged commit 695b8cc into quarkusio:main Sep 17, 2025
60 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.29 - main milestone Sep 17, 2025
@quarkus-bot quarkus-bot bot added kind/enhancement New feature or request and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Sep 17, 2025
@manofthepeace manofthepeace deleted the bumpSurefire branch September 17, 2025 22:32
@gsmet gsmet modified the milestones: 3.29 - main, 3.28.1 Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/amazon-lambda area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/funqy area/kubernetes area/maven area/platform Issues related to definition and interaction with Quarkus Platform kind/enhancement New feature or request triage/flaky-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Please update to surefire plugin 3.5.4
3 participants