Merge branch 'release/0.0.4'

Release: com.io7m.waxmill 0.0.4 Change: Fix issue creating ZFS volumes (Ticket: #28)
io7m · Jul 28, 2020 · eb160f5 · eb160f5
2 parents c1d519e + d07360a
commit eb160f5
Show file tree

Hide file tree

Showing 28 changed files with 557 additions and 54 deletions.
diff --git a/README-CHANGES.xml b/README-CHANGES.xml
@@ -11,7 +11,7 @@
         <c:change date="2020-07-11T00:00:00+00:00" summary="Add the vm-kill command"/>
       </c:changes>
     </c:release>
-    <c:release date="2020-07-25T09:00:46+00:00" ticket-system="com.github.io7m.waxmill" version="0.0.3">
+    <c:release date="2020-07-25T00:00:00+00:00" ticket-system="com.github.io7m.waxmill" version="0.0.3">
       <c:changes>
         <c:change date="2020-07-25T00:00:00+00:00" summary="Add support for interface groups">
           <c:tickets>
@@ -28,13 +28,22 @@
             <c:ticket id="26"/>
           </c:tickets>
         </c:change>
-        <c:change date="2020-07-25T09:00:46+00:00" summary="Automatically create virtual machine filesystems">
+        <c:change date="2020-07-25T00:00:00+00:00" summary="Automatically create virtual machine filesystems">
           <c:tickets>
             <c:ticket id="23"/>
           </c:tickets>
         </c:change>
       </c:changes>
     </c:release>
+    <c:release date="2020-07-28T15:55:45+00:00" ticket-system="com.github.io7m.waxmill" version="0.0.4">
+      <c:changes>
+        <c:change date="2020-07-28T15:55:45+00:00" summary="Fix issue creating ZFS volumes">
+          <c:tickets>
+            <c:ticket id="28"/>
+          </c:tickets>
+        </c:change>
+      </c:changes>
+    </c:release>
   </c:releases>
   <c:ticket-systems>
     <c:ticket-system default="true" id="com.github.io7m.waxmill" url="https://www.github.com/io7m/waxmill/issues/"/>

diff --git a/com.io7m.waxmill.boot/pom.xml b/com.io7m.waxmill.boot/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.boot</artifactId>

diff --git a/com.io7m.waxmill.client.api/pom.xml b/com.io7m.waxmill.client.api/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.client.api</artifactId>

diff --git a/com.io7m.waxmill.client.vanilla/pom.xml b/com.io7m.waxmill.client.vanilla/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.client.vanilla</artifactId>

diff --git a/com.io7m.waxmill.cmdline/pom.xml b/com.io7m.waxmill.cmdline/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.cmdline</artifactId>

diff --git a/com.io7m.waxmill.database.api/pom.xml b/com.io7m.waxmill.database.api/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.database.api</artifactId>

diff --git a/com.io7m.waxmill.database.vanilla/pom.xml b/com.io7m.waxmill.database.vanilla/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.database.vanilla</artifactId>

diff --git a/com.io7m.waxmill.documentation/pom.xml b/com.io7m.waxmill.documentation/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.documentation</artifactId>

diff --git a/com.io7m.waxmill.exceptions/pom.xml b/com.io7m.waxmill.exceptions/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.exceptions</artifactId>

diff --git a/com.io7m.waxmill.locks/pom.xml b/com.io7m.waxmill.locks/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.locks</artifactId>

diff --git a/com.io7m.waxmill.machines/pom.xml b/com.io7m.waxmill.machines/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.machines</artifactId>

diff --git a/com.io7m.waxmill.machines/src/main/java/com/io7m/waxmill/machines/WXMStorageBackends.java b/com.io7m.waxmill.machines/src/main/java/com/io7m/waxmill/machines/WXMStorageBackends.java
@@ -52,11 +52,25 @@ public static Path determineZFSVolumePath(
 
     return machineDirectory
       .resolve(machineId.toString())
-      .resolve(String.format(
-        "disk-%d_%d_%d",
-        Integer.valueOf(deviceID.busID()),
-        Integer.valueOf(deviceID.slotID()),
-        Integer.valueOf(deviceID.functionID())
-      ));
+      .resolve(determineZFSVolumeName(deviceID));
+  }
+
+  /**
+   * Derive the ZFS volume name for the given device ID.
+   *
+   * @param deviceID The device ID
+   *
+   * @return The volume name
+   */
+
+  public static String determineZFSVolumeName(
+    final WXMDeviceSlot deviceID)
+  {
+    return String.format(
+      "disk-%d_%d_%d",
+      Integer.valueOf(deviceID.busID()),
+      Integer.valueOf(deviceID.slotID()),
+      Integer.valueOf(deviceID.functionID())
+    );
   }
 }
diff --git a/com.io7m.waxmill.parser.api/pom.xml b/com.io7m.waxmill.parser.api/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.parser.api</artifactId>

diff --git a/com.io7m.waxmill.process.api/pom.xml b/com.io7m.waxmill.process.api/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.process.api</artifactId>

diff --git a/com.io7m.waxmill.process.posix/pom.xml b/com.io7m.waxmill.process.posix/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.process.posix</artifactId>

diff --git a/com.io7m.waxmill.realize/pom.xml b/com.io7m.waxmill.realize/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.realize</artifactId>

diff --git a/com.io7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/WXMRealizationType.java b/com.io7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/WXMRealizationType.java
@@ -16,6 +16,8 @@
 
 package com.io7m.waxmill.realize;
 
+import com.io7m.waxmill.exceptions.WXMException;
+
 /**
  * A realization operation.
  */
@@ -29,5 +31,6 @@ public interface WXMRealizationType
    * @return A list of instructions
    */
 
-  WXMRealizationInstructions evaluate();
+  WXMRealizationInstructions evaluate()
+    throws WXMException;
 }
diff --git a/com.io7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/WXMRealizations.java b/com.io7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/WXMRealizations.java
@@ -18,6 +18,7 @@
 
 import com.io7m.junreachable.UnimplementedCodeException;
 import com.io7m.waxmill.client.api.WXMClientConfiguration;
+import com.io7m.waxmill.exceptions.WXMException;
 import com.io7m.waxmill.machines.WXMDeviceAHCIDisk;
 import com.io7m.waxmill.machines.WXMDeviceType;
 import com.io7m.waxmill.machines.WXMDeviceVirtioBlockStorage;
@@ -84,6 +85,7 @@ public static WXMRealizationType create(
 
   @Override
   public WXMRealizationInstructions evaluate()
+    throws WXMException
   {
     final var builder = WXMRealizationInstructions.builder();
 
@@ -129,6 +131,7 @@ public WXMRealizationInstructions evaluate()
   private void evaluateAHCIDisk(
     final WXMRealizationInstructions.Builder builder,
     final WXMDeviceAHCIDisk device)
+    throws WXMException
   {
     final var backend = device.backend();
     switch (backend.kind()) {
@@ -163,6 +166,7 @@ private void evaluateFile(
   private void evaluateVirtioBlock(
     final WXMRealizationInstructions.Builder builder,
     final WXMDeviceVirtioBlockStorage device)
+    throws WXMException
   {
     final var backend = device.backend();
     switch (backend.kind()) {
@@ -184,6 +188,7 @@ private void evaluateZFSVolume(
     final WXMRealizationInstructions.Builder builder,
     final WXMDeviceType device,
     final WXMStorageBackendZFSVolume zfs)
+    throws WXMException
   {
     builder.addSteps(
       new WXMZFSVolumeCheck(

diff --git a/...ll.realize/src/main/java/com/io7m/waxmill/realize/internal/WXMRuntimeDirectoryCreate.java b/...ll.realize/src/main/java/com/io7m/waxmill/realize/internal/WXMRuntimeDirectoryCreate.java
@@ -96,12 +96,11 @@ public void execute(
 
     LOG.info("checking if {} is a directory", path);
 
-    if (Files.isDirectory(path)) {
-      LOG.info("{} is a directory", path);
-      return;
-    }
-
     if (Files.exists(path)) {
+      if (Files.isDirectory(path)) {
+        LOG.info("{} is a directory", path);
+        return;
+      }
       throw new WXMException(this.notADirectory(path));
     }
 

diff --git a/...7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/internal/WXMZFSVolumeCheck.java b/...7m.waxmill.realize/src/main/java/com/io7m/waxmill/realize/internal/WXMZFSVolumeCheck.java
@@ -18,6 +18,7 @@
 
 import com.io7m.waxmill.client.api.WXMClientConfiguration;
 import com.io7m.waxmill.exceptions.WXMException;
+import com.io7m.waxmill.exceptions.WXMExceptionUnsatisfiedRequirement;
 import com.io7m.waxmill.machines.WXMDeviceSlot;
 import com.io7m.waxmill.machines.WXMDryRun;
 import com.io7m.waxmill.machines.WXMStorageBackendZFSVolume;
@@ -31,9 +32,11 @@
 
 import java.io.IOException;
 import java.math.BigInteger;
+import java.nio.file.FileStore;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.List;
+import java.util.Locale;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.UUID;
@@ -52,6 +55,7 @@ public final class WXMZFSVolumeCheck implements WXMRealizationStepType
   private final WXMRealizeMessages messages;
   private final WXMStorageBackendZFSVolume zfsVolume;
   private final Path volumePath;
+  private final String volumeCreate;
   private List<WXMProcessDescription> processList;
 
   public WXMZFSVolumeCheck(
@@ -61,6 +65,7 @@ public WXMZFSVolumeCheck(
     final UUID inMachineId,
     final WXMDeviceSlot inSlot,
     final WXMStorageBackendZFSVolume inZFSVolume)
+    throws WXMException
   {
     this.clientConfiguration =
       Objects.requireNonNull(inClientConfiguration, "clientConfiguration");
@@ -75,6 +80,9 @@ public WXMZFSVolumeCheck(
     final var machineId =
       Objects.requireNonNull(inMachineId, "machineId");
 
+    this.volumeCreate =
+      this.determineVolumeCreatePath(machineId);
+
     this.volumePath =
       WXMStorageBackends.determineZFSVolumePath(
         this.clientConfiguration.virtualMachineRuntimeDirectory(),
@@ -86,6 +94,40 @@ public WXMZFSVolumeCheck(
     this.processList = this.processOpt.map(List::of).orElseGet(List::of);
   }
 
+  private String determineVolumeCreatePath(
+    final UUID machineId)
+    throws WXMException
+  {
+    final var base =
+      this.clientConfiguration.virtualMachineRuntimeDirectory()
+        .resolve(machineId.toString());
+
+    final FileStore store;
+    try {
+      store = Files.getFileStore(base);
+    } catch (final IOException e) {
+      throw new WXMException(e);
+    }
+
+    final var type = store.type();
+    if (!"ZFS".equals(type.toUpperCase(Locale.ROOT))) {
+      throw new WXMExceptionUnsatisfiedRequirement(
+        this.messages.format(
+          "errorNotZFS",
+          machineId,
+          base,
+          type
+        )
+      );
+    }
+
+    return String.format(
+      "%s/%s",
+      store.name(),
+      WXMStorageBackends.determineZFSVolumeName(this.slot)
+    );
+  }
+
   private Optional<WXMProcessDescription> makeProcesses()
   {
     return this.zfsVolume.expectedSize()
@@ -94,7 +136,7 @@ private Optional<WXMProcessDescription> makeProcesses()
         .addArguments("create")
         .addArguments("-V")
         .addArguments(size.toString())
-        .addArguments(this.volumePath.toString())
+        .addArguments(this.volumeCreate)
         .build()
       );
   }
@@ -105,7 +147,7 @@ public String description()
     final var expectedSize = this.zfsVolume.expectedSize();
     return this.messages.format(
       "zfsVolumeCheck",
-      this.volumePath,
+      this.volumeCreate,
       expectedSize.map(WXMZFSVolumeCheck::formatSize)
         .orElse("<unspecified>"),
       this.slot

diff --git a/com.io7m.waxmill.realize/src/main/resources/com/io7m/waxmill/realize/internal/Realize.xml b/com.io7m.waxmill.realize/src/main/resources/com/io7m/waxmill/realize/internal/Realize.xml
@@ -44,4 +44,11 @@
 ]]>
   </entry>
 
+  <entry key="errorNotZFS"><![CDATA[The specified directory is not a ZFS filesystem.
+  Machine:         {0}
+  Path:            {1}
+  Filesystem Type: {2}
+]]>
+  </entry>
+
 </properties>
diff --git a/com.io7m.waxmill.serializer.api/pom.xml b/com.io7m.waxmill.serializer.api/pom.xml
@@ -8,7 +8,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.serializer.api</artifactId>

diff --git a/com.io7m.waxmill.strings.api/pom.xml b/com.io7m.waxmill.strings.api/pom.xml
@@ -9,7 +9,7 @@
   <parent>
     <artifactId>com.io7m.waxmill</artifactId>
     <groupId>com.io7m.waxmill</groupId>
-    <version>0.0.3</version>
+    <version>0.0.4</version>
   </parent>
 
   <artifactId>com.io7m.waxmill.strings.api</artifactId>