]> git.proxmox.com Git - qemu-server.git/blobdiff - PVE/QemuMigrate.pm
bwlimit: add parameter for QemuMigrate::phase2
[qemu-server.git] / PVE / QemuMigrate.pm
index d33b8ffa0b0716fa9574035a457edd8552775a4d..4b5813c3c678b995c7bab5e22cb67e4971763400 100644 (file)
@@ -246,6 +246,7 @@ sub prepare {
                if !$plugin->check_connection($sid, $scfg);
        } else {
            # only activate if not shared
+           next if ($volid =~ m/vm-\d+-cloudinit/);
            push @$need_activate, $volid;
        }
     }
@@ -353,7 +354,13 @@ sub sync_disks {
 
            $local_volumes->{$volid}->{ref} = $attr->{referenced_in_config} ? 'config' : 'snapshot';
 
-           die "local cdrom image\n" if $attr->{cdrom};
+           if ($attr->{cdrom}) {
+               if ($volid =~ /vm-\d+-cloudinit/) {
+                   $local_volumes->{$volid}->{ref} = 'generated';
+                   return;
+               }
+               die "local cdrom image\n";
+           }
 
            my ($path, $owner) = PVE::Storage::path($self->{storecfg}, $volid);
 
@@ -394,6 +401,8 @@ sub sync_disks {
                $self->log('info', "found local disk '$vol' (in current VM config)\n");
            } elsif ($ref eq 'snapshot') {
                $self->log('info', "found local disk '$vol' (referenced by snapshot(s))\n");
+           } elsif ($ref eq 'generated') {
+               $self->log('info', "found generated disk '$vol' (in current VM config)\n");
            } else {
                $self->log('info', "found local disk '$vol'\n");
            }
@@ -445,15 +454,26 @@ sub sync_disks {
        foreach my $volid (keys %$local_volumes) {
            my ($sid, $volname) = PVE::Storage::parse_volume_id($volid);
            my $targetsid = $override_targetsid // $sid;
-           if ($self->{running} && $local_volumes->{$volid}->{ref} eq 'config') {
+           my $ref = $local_volumes->{$volid}->{ref};
+           if ($self->{running} && $ref eq 'config') {
                push @{$self->{online_local_volumes}}, $volid;
+           } elsif ($ref eq 'generated') {
+               # skip all generated volumes but queue them for deletion in phase3_cleanup
+               push @{$self->{volumes}}, $volid;
+               next;
            } else {
                next if $rep_volumes->{$volid};
                push @{$self->{volumes}}, $volid;
-               my $insecure = $self->{opts}->{migration_type} eq 'insecure';
+               my $opts = $self->{opts};
+               my $insecure = $opts->{migration_type} eq 'insecure';
                my $with_snapshots = $local_volumes->{$volid}->{snapshots};
+               # use 'migrate' limit for transfer to other node
+               my $bwlimit = PVE::Storage::get_bandwidth_limit('migrate', [$targetsid, $sid], $opts->{bwlimit});
+               # JSONSchema and get_bandwidth_limit use kbps - storage_migrate bps
+               $bwlimit = $bwlimit * 1024 if defined($bwlimit);
+
                PVE::Storage::storage_migrate($self->{storecfg}, $volid, $self->{ssh_info}, $targetsid,
-                                             undef, undef, undef, undef, $insecure, $with_snapshots);
+                                             undef, undef, undef, $bwlimit, $insecure, $with_snapshots);
            }
        }
     };
@@ -582,7 +602,7 @@ sub phase2 {
        elsif ($line =~ m/^spice listens on port (\d+)$/) {
            $spice_port = int($1);
        }
-       elsif ($line =~ m/^storage migration listens on nbd:(localhost|[\d\.]+|\[[\d\.:a-fA-F]+\]):(\d+):exportname=(\S+) volume:(\S+)$/) {
+       elsif ($line =~ m/^storage migration listens on nbd:(localhost|[\d\.]+|\[[\d\.:a-fA-F]+\]):(\d+):exportname=(\S+) volume:(\S+)$/) {
            my $volid = $4;
            my $nbd_uri = "nbd:$1:$2:exportname=$3";
            my $targetdrive = $3;
@@ -641,6 +661,8 @@ sub phase2 {
 
     my $start = time();
 
+    my $opt_bwlimit = $self->{opts}->{bwlimit};
+
     if (defined($self->{online_local_volumes})) {
        $self->{storage_migration} = 1;
        $self->{storage_migration_jobs} = {};
@@ -649,9 +671,14 @@ sub phase2 {
        die "The number of local disks does not match between the source and the destination.\n"
            if (scalar(keys %{$self->{target_drive}}) != scalar @{$self->{online_local_volumes}});
        foreach my $drive (keys %{$self->{target_drive}}){
-           my $nbd_uri = $self->{target_drive}->{$drive}->{nbd_uri};
+           my $target = $self->{target_drive}->{$drive};
+           my $nbd_uri = $target->{nbd_uri};
+           my $source_sid = PVE::Storage::Plugin::parse_volume_id($conf->{$drive});
+           my $target_sid = PVE::Storage::Plugin::parse_volume_id($target->{volid});
+           my $bwlimit = PVE::Storage::get_bandwidth_limit('migrate', [$source_sid, $target_sid], $opt_bwlimit);
+
            $self->log('info', "$drive: start migration to $nbd_uri");
-           PVE::QemuServer::qemu_drive_mirror($vmid, $drive, $nbd_uri, $vmid, undef, $self->{storage_migration_jobs}, 1);
+           PVE::QemuServer::qemu_drive_mirror($vmid, $drive, $nbd_uri, $vmid, undef, $self->{storage_migration_jobs}, 1, undef, $bwlimit);
        }
     }
 
@@ -661,11 +688,21 @@ sub phase2 {
     # load_defaults
     my $defaults = PVE::QemuServer::load_defaults();
 
-    # always set migrate speed (overwrite kvm default of 32m)
-    # we set a very hight default of 8192m which is basically unlimited
-    my $migrate_speed = $defaults->{migrate_speed} || 8192;
-    $migrate_speed = $conf->{migrate_speed} || $migrate_speed;
-    $migrate_speed = $migrate_speed * 1048576;
+    # migrate speed can be set via bwlimit (datacenter.cfg and API) and via the
+    # migrate_speed parameter in qm.conf - take the lower of the two.
+    my $bwlimit = PVE::Storage::get_bandwidth_limit('migrate', undef, $opt_bwlimit) // 0;
+    my $migrate_speed = $conf->{migrate_speed} // $bwlimit;
+    # migrate_speed is in MB/s, bwlimit in KB/s
+    $migrate_speed *= 1024;
+
+    $migrate_speed = ($bwlimit < $migrate_speed) ? $bwlimit : $migrate_speed;
+
+    # always set migrate speed (overwrite kvm default of 32m) we set a very high
+    # default of 8192m which is basically unlimited
+    $migrate_speed ||= ($defaults->{migrate_speed} || 8192) * 1024;
+
+    # qmp takes migrate_speed in B/s.
+    $migrate_speed *= 1024;
     $self->log('info', "migrate_set_speed: $migrate_speed");
     eval {
         PVE::QemuServer::vm_mon_cmd_nocheck($vmid, "migrate_set_speed", value => int($migrate_speed));
@@ -706,7 +743,7 @@ sub phase2 {
        my (undef, $proxyticket) = PVE::AccessControl::assemble_spice_ticket($authuser, $vmid, $self->{node});
 
        my $filename = "/etc/pve/nodes/$self->{node}/pve-ssl.pem";
-       my $subject =  PVE::AccessControl::read_x509_subject_spice($filename);
+       my $subject =  PVE::AccessControl::read_x509_subject_spice($filename);
 
        $self->log('info', "spice client_migrate_info");
 
@@ -991,7 +1028,7 @@ sub phase3_cleanup {
                last if $timer > 50;
                $timer ++;
                usleep(200000);
-           }
+           }
        }
     };