[pve-devel] [PATCH v2 qemu-server 4/6] savevm: periodically print progress

[Stefan Reiter]

Signed-off-by: Stefan Reiter <s.rei...@proxmox.com>
---

v2:
* rename helper
* return tuple
* use 'total-time'
* improve messages

 PVE/QemuConfig.pm | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/PVE/QemuConfig.pm b/PVE/QemuConfig.pm
index 3f4605f..0c4b8fd 100644
--- a/PVE/QemuConfig.pm
+++ b/PVE/QemuConfig.pm
@@ -14,6 +14,7 @@ use PVE::QemuServer;
 use PVE::QemuServer::Machine;
 use PVE::Storage;
 use PVE::Tools;
+use PVE::Format qw(render_bytes render_duration);
 
 use base qw(PVE::AbstractConfig);
 
@@ -280,14 +281,25 @@ sub __snapshot_create_vol_snapshots_hook {
                PVE::Storage::activate_volumes($storecfg, [$snap->{vmstate}]);
 
                mon_cmd($vmid, "savevm-start", statefile => $path);
+               print "saving VM state and RAM\n";
+               my $render_state = sub {
+                   my ($stat) = @_;
+                   my $b = render_bytes($stat->{bytes});
+                   my $t = render_duration($stat->{'total-time'} / 1000);
+                   return ($b, $t);
+               };
                for(;;) {
                    my $stat = mon_cmd($vmid, "query-savevm");
                    if (!$stat->{status}) {
                        die "savevm not active\n";
                    } elsif ($stat->{status} eq 'active') {
                        sleep(1);
+                       my ($b, $t) = $render_state->($stat);
+                       print "$b in $t\n";
                        next;
                    } elsif ($stat->{status} eq 'completed') {
+                       my ($b, $t) = $render_state->($stat);
+                       print "completed savevm in $t, saved $b\n";
                        last;
                    } else {
                        die "query-savevm returned status '$stat->{status}'\n";
-- 
2.20.1



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel