QuarkusComponentTest: skip system config sources by default

[mkouba]

This is a breaking change but I believe that it makes sense and the main goal is to mitigate the problem with Quarkus test profiles described in #48899.

[geoand]

Should there also be a doc entry for this?

[mkouba]

Should there also be a doc entry for this?

It's documented in the javadoc 🤷. We don't have an entry per each feature in the docs.

However, it's a breaking change so we'll need to add a release note.

[geoand]

We don't have an entry per each feature in the docs

No one said we did... But what we do have is entries for stuff that may be useful in general

[mkouba]

We don't have an entry per each feature in the docs

No one said we did... But what we do have is entries for stuff that may be useful in general

All the stuff may be useful in general but I get your point 😉.

[geoand]

👍🏽

[mkouba]

👍🏽

@geoand Done!

[holly-cummins]

Have we tried this with the reproducer for #48899 and friends? I think it's a good idea either way, I'm just curious. :)

[mkouba]

Have we tried this with the reproducer for #48899 and friends? I think it's a good idea either way, I'm just curious. :)

Not yet, I will try that...

[quarkus-bot]

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 56f5850.

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

[mkouba]

Have we tried this with the reproducer for #48899 and friends? I think it's a good idea either way, I'm just curious. :)

Not yet, I will try that...

@holly-cummins so I managed to run the reproducer (even though it's gradle 🙄) and this PR seems to fix the problem.

[github-actions]

🙈 The PR is closed and the preview is expired.

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 56f5850.

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