Skip to content

Conversation

@gsmet
Copy link

@gsmet gsmet commented Nov 26, 2025

I wasn't able to test everything as some testing requires sudo for whatever reason so let's see how CI likes it.

This is to prepare the arrival of UBI 10 images: quarkusio/quarkus-images#313. I started by wanting some test coverage for UBI 10 and this is how it ended up :).

I took Quarkus 3.20 as the new base, which allowed to simplify a lot of stuff.

Note that commits are semantic and should be reviewed one by one.

Chances are it will be the first version fully vetted for Quarkus
I think we really need to push a list of compatible runtimes when adding
a new UBI version, instead of relying on brittle exclusions.
Also prepare the arrival of UBI 10 testing.
@jerboaa jerboaa requested a review from Karm November 26, 2025 18:00
@gsmet
Copy link
Author

gsmet commented Nov 26, 2025

Hmmm, apparently, given the CI matrix, there are additional CI files that require my attention. I'll have a closer look tomorrow.

@jerboaa
Copy link
Collaborator

jerboaa commented Nov 26, 2025

@gsmet I suggest to talk to @Karm about it and how he feels about dropping support for old releases in the test suite. We still have ancient GraalVM version support in there.

@gsmet
Copy link
Author

gsmet commented Nov 27, 2025

@Karm I see one test failing:

Error:    DebugSymbolsTest.debugSymbolsQuarkusContainer:356 Timeout 20s was reached. Empty webpage does not contain string: `"name":"Apple"' ==> expected: <true> but was: <false>

I have been trying to start the app locally and AFAICS the content is here when started locally so I'm not sure what is going on.

@Karm
Copy link
Owner

Karm commented Nov 27, 2025

Hello @gsmet, thank you for visiting us at the Mandrel Integration Testsuite venue :-)

I went over the commits and we do not need these changes at this time except this one:

  • 74d0fbe Provide a list of runtimes instead of excluding some

I like that change a lot and we would like to have it and backport it to the branch https://github.com/Karm/mandrel-integration-tests/tree/testing-more-runtime-images that is used to run tests over at https://github.com/quarkusio/quarkus-images
If you don't mind, I would drop your other commits and integrate this one.

@Karm
Copy link
Owner

Karm commented Nov 27, 2025

Hmm, while testing it I remembered why I did this "such and such is excluded" in the first place.
It was to have a convenient way to print these black squares saying that yes, we know about that runtime, but it was intentionally skipped and why:

image

The current approach seems to make this cumbersome...

@Karm
Copy link
Owner

Karm commented Nov 27, 2025

@gsmet Thanks for looking into the TS. At this moment, I would like to close this PR unmerged and if there are mode UBI10 adjustments needed, I'd do those in #386. I do realize the versioning scheme etc. is confusing and we should do a much better job on the README or CONTRIBUTING docs, e.g. for starters explaining what is this TS actually used for and where. The destiny of the 6 commits in this PR is as follows:

GSmet's commit sha Comment
Update UBI 10 to 10.1 bc08e2f Taken into #386
Use UBI 10 for testing as it will soon be the default e772271 Not yet; would affect too much.
Provide a list of runtimes instead of excluding some 74d0fbe Taken into #386
Reduce testing to only test supported versions 0a59fde Nope, we should support Q 3.2.12.SP1+
Modernize test suite as we only support Quarkus 3.20+ 9836679 Nope, not yet.
Add additional ignore patterns 12398c6 Processed in #385

@Karm Karm closed this Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants