Merge 9ab346e6e7 into 55c36d62ff

src/driver/controller-zfs-generic/index.js

@@ -39,20 +39,26 @@ class ControllerZfsGenericDriver extends ControllerZfsBaseDriver {
       }
       options.idempotent = true;
 
+      options.sudo = _.get(this.options, "zfs.cli.sudoEnabled", false);
+
+      // Run automatic detection first
+      if (typeof this.setZetabyteCustomOptions === "function") {
+        await this.setZetabyteCustomOptions(options);
+      }
+
+      // Manual override comes after automatic detection
+      // Only apply if user explicitly provided non-empty paths
       if (
         this.options.zfs.hasOwnProperty("cli") &&
         this.options.zfs.cli &&
-        this.options.zfs.cli.hasOwnProperty("paths")
+        this.options.zfs.cli.hasOwnProperty("paths") &&
+        this.options.zfs.cli.paths &&
+        Object.keys(this.options.zfs.cli.paths).length > 0
       ) {
+        // User explicitly configured paths - use them
         options.paths = this.options.zfs.cli.paths;
       }
 
-      options.sudo = _.get(this.options, "zfs.cli.sudoEnabled", false);
-
-      if (typeof this.setZetabyteCustomOptions === "function") {
-        await this.setZetabyteCustomOptions(options);
-      }
-
       return new Zetabyte(options);
     });
   }

src/driver/freenas/ssh.js

@@ -71,20 +71,26 @@ class FreeNASSshDriver extends ControllerZfsBaseDriver {
       options.executor = new ZfsSshProcessManager(sshClient);
       options.idempotent = true;
 
+      options.sudo = _.get(this.options, "zfs.cli.sudoEnabled", false);
+
+      // Run automatic detection first
+      if (typeof this.setZetabyteCustomOptions === "function") {
+        await this.setZetabyteCustomOptions(options);
+      }
+
+      // Manual override comes after automatic detection
+      // Only apply if user explicitly provided non-empty paths
       if (
         this.options.zfs.hasOwnProperty("cli") &&
         this.options.zfs.cli &&
-        this.options.zfs.cli.hasOwnProperty("paths")
+        this.options.zfs.cli.hasOwnProperty("paths") &&
+        this.options.zfs.cli.paths &&
+        Object.keys(this.options.zfs.cli.paths).length > 0
       ) {
+        // User explicitly configured paths - use them
         options.paths = this.options.zfs.cli.paths;
       }
 
-      options.sudo = _.get(this.options, "zfs.cli.sudoEnabled", false);
-
-      if (typeof this.setZetabyteCustomOptions === "function") {
-        await this.setZetabyteCustomOptions(options);
-      }
-
       return new Zetabyte(options);
     });
   }
@@ -112,22 +118,33 @@ class FreeNASSshDriver extends ControllerZfsBaseDriver {
     if (!options.hasOwnProperty("paths")) {
       const majorMinor = await this.getSystemVersionMajorMinor();
       const isScale = await this.getIsScale();
+      
       if (!isScale && Number(majorMinor) >= 12) {
+        // TrueNAS CORE/Enterprise version 12+
         options.paths = {
           zfs: "/usr/local/sbin/zfs",
           zpool: "/usr/local/sbin/zpool",
           sudo: "/usr/local/bin/sudo",
           chroot: "/usr/sbin/chroot",
         };
-      }
-
-      if (isScale && Number(majorMinor) >= 25) {
+      } else if (isScale) {
+        if (Number(majorMinor) >= 25) {
+          // TrueNAS SCALE version 25+ (paths changed to /usr/sbin in 25.04)
           options.paths = {
             zfs: "/usr/sbin/zfs",
             zpool: "/usr/sbin/zpool",
             sudo: "/usr/bin/sudo",
             chroot: "/usr/sbin/chroot",
           };
+        } else {
+          // TrueNAS SCALE versions before 25
+          options.paths = {
+            zfs: "/usr/local/sbin/zfs",
+            zpool: "/usr/local/sbin/zpool",
+            sudo: "/usr/bin/sudo",
+            chroot: "/usr/sbin/chroot",
+          };
+        }
       }
     }
   }