Make the maven build task compatible with parallel builds

[essobedo]

fixes #16932

Motivation

So far, due to the fact that System properties can be set by the build task of the maven plugin, we could end up with build conflicts when used in parallel mode. The goal of this enhancement is to ensure that we can use parallel builds without getting conflicts.

Modifications

• Relies on the runtime default profile to isolate the value of the build profile since the class ProfileManager is loaded with different QuarkusClassLoaders
• Sets the runtime default profile using the value set in the properties of the plugin
• Prints a warning if the parallel build is enabled and system properties are used
• Adds the ability to launch a build of an integration test with parallel mode enabled
• Adds an integration test for it

[quarkus-bot]

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

Failing Jobs - Building 95beb0a

Status	Name	Step	Test failures	Logs	Raw logs
✖	Initial JDK 11 Build	Build	⚠️ Check →	Logs	Raw logs

[quarkus-bot]

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

🚫 This workflow run has been cancelled.

Failing Jobs - Building b015f29

⚠️ Artifacts of the workflow run were not available thus the report misses some details.

Status	Name	Step	Test failures	Logs	Raw logs
⌛	Devtools Tests - JDK 11 Windows	Build	⚠️ Check →	Logs	Raw logs
⌛	Gradle Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
⌛	Gradle Tests - JDK 11 Windows	Build	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 11 Windows	Build	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 16	Build	⚠️ Check →	Logs	Raw logs
⌛	Maven Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
⌛	Maven Tests - JDK 11 Windows	Build	Test failures	Logs	Raw logs
⌛	MicroProfile TCKs Tests	Verify	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Amazon	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Cache		⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data1	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data2	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data3	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data4	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data5	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data6	Build	⚠️ Check →	Logs	Raw logs
✖	Native Tests - Data7	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - HTTP		⚠️ Check →	Logs	Raw logs
✖	Native Tests - Main	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Messaging1	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Messaging2	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc1		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc2		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc3		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc4		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security1	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security2	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security3	Build	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Spring		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Windows - hibernate-validator		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - gRPC		⚠️ Check →	Logs	Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ Maven Tests - JDK 11 Windows #

📦 integration-tests/maven

✖ io.quarkus.maven.it.BuildIT.testModuleWithOverriddenBuildProfile line 98 - More details - Source on GitHub

[quarkus-bot]

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

Failing Jobs - Building 7f5079a

Status	Name	Step	Test failures	Logs	Raw logs
✖	Initial JDK 11 Build	Build	⚠️ Check →	Logs	Raw logs

[quarkus-bot]

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

Failing Jobs - Building 4f6dc1d

Status	Name	Step	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
✔️	JVM Tests - JDK 16
✖	Maven Tests - JDK 11	Build	Test failures	Logs	Raw logs
✖	Maven Tests - JDK 11 Windows	Build	Test failures	Logs	Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ Maven Tests - JDK 11 #

📦 integration-tests/maven

✖ io.quarkus.maven.it.BuildIT.testModuleWithOverriddenBuildProfile line 98 - More details - Source on GitHub

---

⚙️ Maven Tests - JDK 11 Windows #

📦 integration-tests/maven

✖ io.quarkus.maven.it.BuildIT.testModuleWithOverriddenBuildProfile line 98 - More details - Source on GitHub

[essobedo]

Removed as it is actually dead code

[essobedo]

@aloubyansky @famod Hi, WDYT of this PR? It is the best compromise I could find as it does the job and it doesn't brake anything. To be honest, I'm not really fan of relying on the runtime default profile, I would rather prefer having a specific static field for that but if I do so I break the test io.quarkus.maven.it.BuildIT.testModuleWithOverriddenBuildProfile

[aloubyansky]

Looks good @essobedo Thanks!

[quarkus-bot]

Failing Jobs - Building c44cdc8

Status	Name	Step	Test failures	Logs	Raw logs
✖	Native Tests - Misc4	Build	⚠️ Check →	Logs	Raw logs

[essobedo]

FYI I don't believe that this PR is the root cause of the build failure since it is an IT of Gradle in native mode and this PR doesn't affect the Gradle related code.

[essobedo]

@aloubyansky Could you please relaunch the build if needed?

[famod]

@aloubyansky WDYT about backporting (to 2.0)?

[essobedo]

@aloubyansky WDYT about backporting (to 2.0)?

Let me know if you want a back port, I can provide it.

[essobedo]

BTW I did not forget to provide some doc about what has been done at maven plugin level, it will be my next contribution. I need to finish something else first

[famod]

Let me know if you want a back port, I can provide it.

Currently, this would merge just fine to the 2.0 branch. But thanks for your offer!

BTW what is the target version of the main now ? 2.1 ?

Yes, see the milestone assignment (top right).