From 8eceb712e2864498eaa29c22828a3942dd5524fb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Galder=20Zamarren=CC=83o?= Date: Tue, 5 Sep 2023 10:22:14 +0200 Subject: [PATCH] Enable native heap dump monitoring by default --- .../deployment/pkg/steps/NativeImageBuildStep.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java b/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java index 1995c8d6b0533..e409a07245b23 100644 --- a/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java +++ b/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java @@ -890,13 +890,15 @@ public NativeImageInvokerInfo build() { nativeImageArgs.add("-H:+AllowVMInspection"); } + List monitoringOptions = new ArrayList<>(); + monitoringOptions.add(NativeConfig.MonitoringOption.HEAPDUMP); if (nativeConfig.monitoring().isPresent()) { - List monitoringOptions = nativeConfig.monitoring().get(); - if (!monitoringOptions.isEmpty()) { - nativeImageArgs.add("--enable-monitoring=" + monitoringOptions.stream() - .map(o -> o.name().toLowerCase(Locale.ROOT)).collect(Collectors.joining(","))); - } + monitoringOptions.addAll(nativeConfig.monitoring().get()); } + nativeImageArgs.add("--enable-monitoring=" + monitoringOptions.stream() + .distinct() + .map(o -> o.name().toLowerCase(Locale.ROOT)).collect(Collectors.joining(","))); + if (nativeConfig.autoServiceLoaderRegistration()) { addExperimentalVMOption(nativeImageArgs, "-H:+UseServiceLoaderFeature"); //When enabling, at least print what exactly is being added: