FISH-12969 Payara Server Startup Fails in Apache NetBeans IDE Due to the CRaCCheckpointTo Option

Author: jGauravGupta

enterprise/payara.tooling/nbproject/org-netbeans-modules-payara-tooling.sig

@@ -1249,7 +1249,10 @@ meth public java.util.Optional<java.lang.String> getVM()
 meth public java.util.Optional<java.lang.String> getVendor()
 meth public short getMajor()
 meth public static boolean isCorrectJDK(java.util.Optional<org.netbeans.modules.payara.tooling.data.JDKVersion>,java.util.Optional<org.netbeans.modules.payara.tooling.data.JDKVersion>)
+ anno 0 java.lang.Deprecated()
 meth public static boolean isCorrectJDK(org.netbeans.modules.payara.tooling.data.JDKVersion,java.util.Optional<java.lang.String>,java.util.Optional<org.netbeans.modules.payara.tooling.data.JDKVersion>,java.util.Optional<org.netbeans.modules.payara.tooling.data.JDKVersion>)
+ anno 0 java.lang.Deprecated()
+meth public boolean isOptionSupported(org.netbeans.modules.payara.tooling.server.parser.JvmConfigReader.JvmOption,java.lang.String)
 meth public static org.netbeans.modules.payara.tooling.data.JDKVersion getDefaultPlatformVersion()
 meth public static org.netbeans.modules.payara.tooling.data.JDKVersion toValue(java.lang.String)
 meth public static org.netbeans.modules.payara.tooling.data.JDKVersion toValue(java.lang.String,java.lang.String)

enterprise/payara.tooling/src/org/netbeans/modules/payara/tooling/data/JDKVersion.java

@@ -19,6 +19,7 @@
 package org.netbeans.modules.payara.tooling.data;
 
 import java.util.Optional;
+import org.netbeans.modules.payara.tooling.server.parser.JvmConfigReader;
 
 public final class JDKVersion {
 
@@ -283,26 +284,75 @@ public static JDKVersion getDefaultPlatformVersion() {
         return IDE_JDK_VERSION;
     }
 
-    public static boolean isCorrectJDK(JDKVersion jdkVersion, Optional<String> vendorOrVM, Optional<JDKVersion> minVersion, Optional<JDKVersion> maxVersion) {
+    public static boolean isOptionSupported(
+            JDKVersion jdkVersion,
+            JvmConfigReader.JvmOption jvmOption,
+            String javaHome) {
+
+        boolean correctJDK = isCorrectJDK(
+                jdkVersion,
+                jvmOption.vendorOrVM,
+                jvmOption.minVersion,
+                jvmOption.maxVersion);
+
+        if (correctJDK
+                && jvmOption.option != null
+                && jvmOption.option.matches("^-XX:[+-]?CRaC.*")) {
+
+            correctJDK = jdkVersion.isCRaCSupported(javaHome);
+        }
+        return correctJDK;
+    }
+
+    @Deprecated
+    public static boolean isCorrectJDK(
+            JDKVersion jdkVersion,
+            Optional<String> vendorOrVM,
+            Optional<JDKVersion> minVersion,
+            Optional<JDKVersion> maxVersion) {
+
         boolean correctJDK = true;
 
         if (vendorOrVM.isPresent()) {
-            correctJDK = jdkVersion.getVendor().map(vendor -> vendor.contains(vendorOrVM.get())).orElse(false)
-                    || jdkVersion.getVM().map(vm -> vm.contains(vendorOrVM.get())).orElse(false);
+            correctJDK
+                    = jdkVersion.getVendor()
+                            .map(v -> v.contains(vendorOrVM.get()))
+                            .orElse(false)
+                    || jdkVersion.getVM()
+                            .map(vm -> vm.contains(vendorOrVM.get()))
+                            .orElse(false);
         }
+
         if (correctJDK && minVersion.isPresent()) {
             correctJDK = jdkVersion.ge(minVersion.get());
         }
+
         if (correctJDK && maxVersion.isPresent()) {
             correctJDK = jdkVersion.le(maxVersion.get());
         }
+
         return correctJDK;
     }
 
+    @Deprecated
     public static boolean isCorrectJDK(Optional<JDKVersion> minVersion, Optional<JDKVersion> maxVersion) {
         return isCorrectJDK(IDE_JDK_VERSION, Optional.empty(), minVersion, maxVersion);
     }
 
+    /**
+     * Checks whether the given JDK installation supports CRaC by verifying the
+     * presence of the lib/criu directory.
+     *
+     * @param javaHome Java home directory to check
+     * @return true if CRaC-enabled JDK
+     */
+    private boolean isCRaCSupported(String javaHome) {
+        return Optional.ofNullable(javaHome)
+                .map(home -> new java.io.File(home, "lib/criu"))
+                .map(java.io.File::exists)
+                .orElse(false);
+    }
+
     static {
         initialize();
     }

enterprise/payara.tooling/src/org/netbeans/modules/payara/tooling/server/ServerTasks.java

@@ -157,11 +157,19 @@ public static ResultProcess startServer(PayaraServer server,
             }
         }
 
-        JDKVersion javaVersion = args.getJavaVersion() == null ? JDKVersion.getDefaultPlatformVersion() : args.getJavaVersion() ;
+        JDKVersion javaVersion = args.getJavaVersion() == null ? JDKVersion.getDefaultPlatformVersion() : args.getJavaVersion();
+        String selectedJavaHome
+                = args.getJavaHome() != null
+                ? args.getJavaHome()
+                : System.getProperty("java.home");
         List<String> optList
                 = jvmConfigReader.getJvmOptions()
                         .stream()
-                        .filter(fullOption -> JDKVersion.isCorrectJDK(javaVersion, fullOption.vendorOrVM, fullOption.minVersion, fullOption.maxVersion))
+                        .filter(fullOption
+                                -> JDKVersion.isOptionSupported(
+                                javaVersion,
+                                fullOption,
+                                selectedJavaHome))
                         .map(fullOption -> fullOption.option)
                         .collect(toList());