Skip to content

Commit e724332

Browse files
authored
Merge pull request quarkusio#35736 from galderz/topic.0905.enable-heapdump
Enable native heap dump monitoring by default
2 parents 3cafdff + 8eceb71 commit e724332

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -890,13 +890,15 @@ public NativeImageInvokerInfo build() {
890890
addExperimentalVMOption(nativeImageArgs, "-H:+AllowVMInspection");
891891
}
892892

893+
List<NativeConfig.MonitoringOption> monitoringOptions = new ArrayList<>();
894+
monitoringOptions.add(NativeConfig.MonitoringOption.HEAPDUMP);
893895
if (nativeConfig.monitoring().isPresent()) {
894-
List<NativeConfig.MonitoringOption> monitoringOptions = nativeConfig.monitoring().get();
895-
if (!monitoringOptions.isEmpty()) {
896-
nativeImageArgs.add("--enable-monitoring=" + monitoringOptions.stream()
897-
.map(o -> o.name().toLowerCase(Locale.ROOT)).collect(Collectors.joining(",")));
898-
}
896+
monitoringOptions.addAll(nativeConfig.monitoring().get());
899897
}
898+
nativeImageArgs.add("--enable-monitoring=" + monitoringOptions.stream()
899+
.distinct()
900+
.map(o -> o.name().toLowerCase(Locale.ROOT)).collect(Collectors.joining(",")));
901+
900902
if (nativeConfig.autoServiceLoaderRegistration()) {
901903
addExperimentalVMOption(nativeImageArgs, "-H:+UseServiceLoaderFeature");
902904
//When enabling, at least print what exactly is being added:

0 commit comments

Comments
 (0)