Merge branch 'feature/pools'

Author: mnlipp

.markdownlint.yaml

@@ -1,4 +1,4 @@
-# See https://github.com/DavidAnson/markdownlint/blob/main/schema/.markdownlint.yaml
+# See [rules](https://github.com/DavidAnson/markdownlint/blob/main/schema/.markdownlint.yaml)
 
 # Default state for all rules
 default: true
@@ -27,12 +27,4 @@ MD036: false
 
 # MD043/required-headings : Required heading structure : 
 # https://github.com/DavidAnson/markdownlint/blob/v0.37.4/doc/md043.md
-MD043:
-  # List of headings
-  headings:   [
-      "# Head",
-      "## Item",
-      "### Detail"
-  ]
-  # Match case of headings
-  match_case: false
+MD043: false

org.jdrupes.vmoperator.common/src/org/jdrupes/vmoperator/common/K8sDynamicModelsBase.java

@@ -62,7 +62,7 @@ public K8sDynamicModelsBase(Class<T> itemClass, Gson delegate,
             } catch (InstantiationException | IllegalAccessException
                     | IllegalArgumentException | InvocationTargetException
                     | NoSuchMethodException | SecurityException exc) {
-                throw new IllegalArgumentException(exc);
+                throw new IllegalArgumentException(exc); // NOPMD
             }
         }
     }

org.jdrupes.vmoperator.runner.qemu/src/org/jdrupes/vmoperator/runner/qemu/Runner.java

@@ -192,7 +192,7 @@
  */
 @SuppressWarnings({ "PMD.ExcessiveImports", "PMD.AvoidPrintStackTrace",
     "PMD.DataflowAnomalyAnalysis", "PMD.TooManyMethods",
-    "PMD.CouplingBetweenObjects" })
+    "PMD.CouplingBetweenObjects", "PMD.TooManyFields" })
 public class Runner extends Component {
 
     private static final String QEMU = "qemu";

org.jdrupes.vmoperator.util/src/org/jdrupes/vmoperator/util/DataPath.java

@@ -18,6 +18,7 @@
 
 package org.jdrupes.vmoperator.util;
 
+import java.lang.reflect.Array;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
@@ -157,11 +158,12 @@ public static <T> T deepCopy(T object) {
             return (T) copy;
         }
         if (object.getClass().isArray()) {
-            var copy = new ArrayList<>();
-            for (var item : (Object[]) object) {
-                copy.add(deepCopy(item));
+            var copy = Array.newInstance(object.getClass().getComponentType(),
+                Array.getLength(object));
+            for (int i = 0; i < Array.getLength(object); i++) {
+                Array.set(copy, i, deepCopy(Array.get(object, i)));
             }
-            return (T) copy.toArray();
+            return (T) copy;
         }
         if (object instanceof Cloneable) {
             try {

org.jdrupes.vmoperator.util/test/org/jdrupes/vmoperator/util/DataPathTests.java

@@ -0,0 +1,17 @@
+package org.jdrupes.vmoperator.util;
+
+import static org.junit.jupiter.api.Assertions.*;
+import org.junit.jupiter.api.Test;
+
+class DataPathTests {
+
+    @Test
+    void testArray() {
+        int[] orig
+            = { Integer.valueOf(1), Integer.valueOf(2), Integer.valueOf(3) };
+        var copy = DataPath.deepCopy(orig);
+        for (int i = 0; i < orig.length; i++) {
+            assertEquals(orig[i], copy[i]);
+        }
+    }
+}