Skip to content

Improve AOT docs#52646

Merged
gsmet merged 4 commits intoquarkusio:mainfrom
geoand:aot-docs
Feb 18, 2026
Merged

Improve AOT docs#52646
gsmet merged 4 commits intoquarkusio:mainfrom
geoand:aot-docs

Conversation

@geoand
Copy link
Contributor

@geoand geoand commented Feb 18, 2026

No description provided.

@quarkus-bot
Copy link

quarkus-bot bot commented Feb 18, 2026

Thanks for your pull request!

Your pull request does not follow our editorial rules. Could you have a look?

  • description should not be empty, describe your intent or provide links to the issues this PR is fixing (using Fixes #NNNNN) or changelogs

This message is automatically generated by a bot.

@geoand
Copy link
Contributor Author

geoand commented Feb 18, 2026

@gsmet I have no idea if what I did for the gradle commands works in adoc

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

🙈 The PR is closed and the preview is expired.

@gsmet
Copy link
Member

gsmet commented Feb 18, 2026

Let me have a look.

@gsmet
Copy link
Member

gsmet commented Feb 18, 2026

From what I can see, it looks good when generating in tree.

There's one thing though, in things like:

The quarkus.package.jar.aot.phase configuration controls when training occurs:

AUTO (default): Uses INTEGRATION_TESTS for Leyden AOT

BUILD: Generates the cache during build (limited training, faster builds)

INTEGRATION_TESTS: Generates during test execution (better optimization)

We should use the hyphenated version to be consistent with the doc and how we deal with enum values in config in general, so: auto, build, integration-tests (as in the generated config doc).

And this for all occurrences. I can do it if you want, I'm looking for easy tasks. But maybe you are too :)

@geoand
Copy link
Contributor Author

geoand commented Feb 18, 2026

Feel free to take it if you like :)

In particular, we only include the AOT doc now, instead of the whole
package doc.
@gsmet
Copy link
Member

gsmet commented Feb 18, 2026

I pushed some changes in an additional commit.

One thing that is different is that we now only show the AOT config doc, instead of the whole package config doc.
That means we don't show the appcds config doc anymore but I think it's fine. It's deprecated and it's still visible in the page listing all the properties.

If we really want it, I can add it in the AppCDS section.

@geoand
Copy link
Contributor Author

geoand commented Feb 18, 2026

That's great, thanks!

@geoand geoand marked this pull request as ready for review February 18, 2026 14:34
@quarkus-bot
Copy link

quarkus-bot bot commented Feb 18, 2026

Status for workflow Quarkus Documentation CI

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

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

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Feb 18, 2026
@quarkus-bot
Copy link

quarkus-bot bot commented Feb 18, 2026

Status for workflow Quarkus CI

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

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
Native Tests - Main Upload build reports ⚠️ Check → Logs Raw logs 🔍

You can consult the Develocity build scans.


Flaky tests - Develocity

⚙️ JVM Tests - JDK 17

📦 extensions/smallrye-reactive-messaging-kafka/deployment

io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingTestCase.testContinuousTestingScenario2 - History

  • org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/apicurio/apicurio-registry:3.1.7 - java.util.concurrent.CompletionException
Details
java.util.concurrent.CompletionException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/apicurio/apicurio-registry:3.1.7
	at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315)
	at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1807)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1796)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655)

⚙️ JVM Tests - JDK 25 Semeru

📦 extensions/smallrye-reactive-messaging/deployment

io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector - History

  • Expecting actual: ["-4","-5","-6","-7","-8","-9","-10","-11"] to start with: ["-3", "-4", "-5", "-6"] - java.lang.AssertionError
Details
java.lang.AssertionError: 

Expecting actual:
  ["-4","-5","-6","-7","-8","-9","-10","-11"]
to start with:
  ["-3", "-4", "-5", "-6"]

	at io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector(ConnectorChangeTest.java:36)

@gsmet gsmet merged commit 30d3e7d into quarkusio:main Feb 18, 2026
84 of 85 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.34 - main milestone Feb 18, 2026
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Feb 18, 2026
@cescoffier cescoffier moved this to Done in WG - AOT Support Feb 23, 2026
@cescoffier
Copy link
Member

🤖 Automatically associated with the WG - AOT Support Working Group based on AI classification.

@gsmet gsmet modified the milestones: 3.34 - main, 3.32.1 Feb 24, 2026
@geoand geoand deleted the aot-docs branch February 26, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants