Skip to content

Commit

Permalink
Recommend quarkus.jib.jvm-additional-arguments rather than quarkus.ji…
Browse files Browse the repository at this point in the history
…b.jvm-arguments in docs

I think that setting quarkus.jib.jvm-arguments would lead to the default
(-Djava.util.logging.manager=org.jboss.logmanager.LogManager) being
replaced, which would mess with logging, so that's probably not
something we want.

(cherry picked from commit dacae71)
  • Loading branch information
yrodiere authored and gsmet committed Jan 18, 2024
1 parent 837f0e2 commit d5d162e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/src/main/asciidoc/container-image.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -47,14 +47,14 @@ For example, the presence of `src/main/jib/foo/bar` would result in `/foo/bar`

There are cases where the built container image may need to have Java debugging conditionally enabled at runtime.

When the base image has not been changed (and therefore `ubi8/openjdk-11-runtime`, `ubi8/openjdk-17-runtime`, or `ubi8/openjdk-21-runtime` is used), then the `quarkus.jib.jvm-arguments` configuration property can be used in order to
When the base image has not been changed (and therefore `ubi8/openjdk-11-runtime`, `ubi8/openjdk-17-runtime`, or `ubi8/openjdk-21-runtime` is used), then the `quarkus.jib.jvm-additional-arguments` configuration property can be used in order to
make the JVM listen on the debug port at startup.

The exact configuration is:

[source,properties]
----
quarkus.jib.jvm-arguments=-agentlib:jdwp=transport=dt_socket\\,server=y\\,suspend=n\\,address=*:5005
quarkus.jib.jvm-additional-arguments=-agentlib:jdwp=transport=dt_socket\\,server=y\\,suspend=n\\,address=*:5005
----

Other base images might provide launch scripts that enable debugging when an environment variable is set, in which case you would set than environment variable when launching the container.
Expand Down

0 comments on commit d5d162e

Please sign in to comment.