[PATCH proxmox-datacenter-manager v2 1/2] cli: add target-vmid parameter to remote_migrate_qemu

[PATCH proxmox-datacenter-manager v2 1/2] cli: add target-vmid parameter to remote_migrate_qemu

[Maximiliano Sandoval]

Adds a --target-vmid parameter to `proxmox-datacenter-manager-client pve
qemu remote-migrate`.

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
---
Differences from v1:

 - Add the optional argument to pve lxc remote-migrate too

 cli/client/src/pve.rs | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/cli/client/src/pve.rs b/cli/client/src/pve.rs
index 16289986..5c2a1abd 100644
--- a/cli/client/src/pve.rs
+++ b/cli/client/src/pve.rs
@@ -546,6 +546,10 @@ async fn migrate_qemu(
             },
             vmid: { schema: VMID_SCHEMA },
             target: { schema: REMOTE_ID_SCHEMA },
+            "target-vmid": {
+                optional: true,
+                schema: VMID_SCHEMA,
+            },
             delete: {
                 description: "Delete the original VM and related data after successful migration.",
                 optional: true,
@@ -587,6 +591,7 @@ async fn remote_migrate_qemu(
     node: Option<String>,
     vmid: u32,
     target: String,
+    target_vmid: Option<u32>,
     online: Option<bool>,
     delete: Option<bool>,
     map_storage: Vec<String>,
@@ -603,6 +608,9 @@ async fn remote_migrate_qemu(
     if let Some(bwlimit) = bwlimit {
         params = params.bwlimit(bwlimit);
     }
+    if let Some(target_vmid) = target_vmid {
+        params = params.target_vmid(target_vmid);
+    }
     for mapping in map_storage {
         let pos = mapping
             .find(':')
-- 
2.47.3

[PATCH proxmox-datacenter-manager v2 2/2] cli: add target-vmid parameter to remote_migrate_lxc

[Maximiliano Sandoval]

Adds a --target-vmid parameter to `proxmox-datacenter-manager-client pve
lxc remote-migrate`.

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
---
 cli/client/src/pve.rs | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/cli/client/src/pve.rs b/cli/client/src/pve.rs
index 5c2a1abd..19bd0980 100644
--- a/cli/client/src/pve.rs
+++ b/cli/client/src/pve.rs
@@ -918,6 +918,10 @@ async fn migrate_lxc(
             },
             vmid: { schema: VMID_SCHEMA },
             target: { schema: REMOTE_ID_SCHEMA },
+            "target-vmid": {
+                optional: true,
+                schema: VMID_SCHEMA,
+            },
             delete: {
                 description: "Delete the original VM and related data after successful migration.",
                 optional: true,
@@ -968,6 +972,7 @@ async fn remote_migrate_lxc(
     node: Option<String>,
     vmid: u32,
     target: String,
+    target_vmid: Option<u32>,
     online: Option<bool>,
     delete: Option<bool>,
     map_storage: Vec<String>,
@@ -986,6 +991,9 @@ async fn remote_migrate_lxc(
     if let Some(bwlimit) = bwlimit {
         params = params.bwlimit(bwlimit);
     }
+    if let Some(target_vmid) = target_vmid {
+        params = params.target_vmid(target_vmid);
+    }
     if restart {
         params = params.restart(true, timeout.map(Duration::from_secs));
     }
-- 
2.47.3

applied: [PATCH proxmox-datacenter-manager v2 1/2] cli: add target-vmid parameter to remote_migrate_qemu

[Lukas Wagner]

On Tue Feb 3, 2026 at 9:17 AM CET, Maximiliano Sandoval wrote:
> Adds a --target-vmid parameter to `proxmox-datacenter-manager-client pve
> qemu remote-migrate`.
>
> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
> ---
> Differences from v1:
>
>  - Add the optional argument to pve lxc remote-migrate too
>

applied both patches, thank you!

I slightly augmented the commit message with the info that the API
already supported this parameter and that this only adds the parameter
to the CLI client - as discussed off-list.