Clone to vm from template onto another storagepool

builder/proxmox/clone/builder.go

@@ -191,6 +191,21 @@ func (*cloneVMCreator) Create(vmRef *proxmoxapi.VmRef, vmConfig proxmoxapi.Confi
 		}
 	}
 
+	storagePool := c.TargetStoragePool
+	if storagePool == "" && len(c.Disks) > 0 {
+		storagePool = c.Disks[0].StoragePool
+	}
+	if fullClone == 1 && storagePool != "" {
+		if vmConfig.QemuDisks == nil {
+			vmConfig.QemuDisks = proxmoxapi.QemuDevices{}
+		}
+		if vmConfig.QemuDisks[0] == nil {
+			vmConfig.QemuDisks[0] = make(map[string]interface{})
+		}
+		vmConfig.QemuDisks[0]["storage"] = storagePool
+		ui.Say(fmt.Sprintf("Using target storage pool: %s", storagePool))
+	}
+
 	err := vmConfig.CloneVm(sourceVmr, vmRef, client)
 	if err != nil {
 		return err

builder/proxmox/clone/config.go

@@ -30,6 +30,10 @@ type Config struct {
 	CloneVMID int `mapstructure:"clone_vm_id" required:"true"`
 	// Whether to run a full or shallow clone from the base clone_vm. Defaults to `true`.
 	FullClone config.Trilean `mapstructure:"full_clone" required:"false"`
+	// Name of the Proxmox storage pool to store the cloned VM disks on.
+	// If not given, the source template's storage is used, or the storage_pool of the first defined disk.
+	// This setting only applies to full clones.
+	TargetStoragePool string `mapstructure:"target_storage_pool" required:"false"`
 
 	// Set nameserver IP address(es) via Cloud-Init.
 	// If not given, the same setting as on the host is used.
@@ -89,6 +93,9 @@ func (c *Config) Prepare(raws ...interface{}) ([]string, []string, error) {
 	if c.CloneVMID != 0 && (c.CloneVMID < 100 || c.CloneVMID > 999999999) {
 		errs = packersdk.MultiErrorAppend(errs, errors.New("clone_vm_id must be in range 100-999999999"))
 	}
+	if c.FullClone.False() && c.TargetStoragePool != "" {
+		warnings = append(warnings, "target_storage_pool is only used for full clones and will be ignored when full_clone is false")
+	}
 
 	// Check validity of given IP addresses
 	if c.Nameserver != "" {

builder/proxmox/clone/config_test.go

@@ -272,3 +272,37 @@ func TestIpconfig(t *testing.T) {
 		})
 	}
 }
+
+func TestTargetStoragePool(t *testing.T) {
+	t.Run("target_storage_pool with full clone", func(t *testing.T) {
+		cfg := mandatoryConfig(t)
+		cfg["target_storage_pool"] = "local-lvm"
+
+		var c Config
+		_, warnings, err := c.Prepare(&c, cfg)
+		if err != nil {
+			t.Fatalf("unexpected failure: %s", err)
+		}
+		if len(warnings) != 0 {
+			t.Fatalf("unexpected warnings: %v", warnings)
+		}
+	})
+
+	t.Run("target_storage_pool with linked clone warning", func(t *testing.T) {
+		cfg := mandatoryConfig(t)
+		cfg["target_storage_pool"] = "local-lvm"
+		cfg["full_clone"] = false
+
+		var c Config
+		_, warnings, err := c.Prepare(&c, cfg)
+		if err != nil {
+			t.Fatalf("unexpected failure: %s", err)
+		}
+		if len(warnings) != 1 {
+			t.Fatalf("expected one warning, got %d (%v)", len(warnings), warnings)
+		}
+		if !strings.Contains(warnings[0], "target_storage_pool") {
+			t.Fatalf("expected target_storage_pool warning, got %q", warnings[0])
+		}
+	})
+}

docs-partials/builder/proxmox/clone/Config-not-required.mdx

@@ -2,6 +2,10 @@
 
 - `full_clone` (boolean) - Whether to run a full or shallow clone from the base clone_vm. Defaults to `true`.
 
+- `target_storage_pool` (string) - Name of the Proxmox storage pool to store the cloned VM disks on.
+  If not given, the source template's storage is used.
+  This setting only applies to full clones.
+
 - `nameserver` (string) - Set nameserver IP address(es) via Cloud-Init.
   If not given, the same setting as on the host is used.