[virt-manager RFC PATCH 2/2] details: snapshots: Warn of saved state on creation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Add a confirmation dialog when creating a new snapshot if there is saved
memory state so the user is aware that memory state will not be part of
the snapshot.

Signed-off-by: Michael Weiser <michael.weiser@xxxxxx>
Suggested-by: Cole Robinson <crobinso@xxxxxxxxxx>
---
 virtManager/details/snapshots.py | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/virtManager/details/snapshots.py b/virtManager/details/snapshots.py
index bc5b915a..c2aecd93 100644
--- a/virtManager/details/snapshots.py
+++ b/virtManager/details/snapshots.py
@@ -633,6 +633,17 @@ class vmmSnapshotPage(vmmGObjectUI):
             self._snapshot_new = vmmSnapshotNew(self.vm)
             self._snapshot_new.connect("snapshot-created",
                     self._snapshot_created_cb)
+        if self.vm.has_managed_save():
+            result = self.err.ok_cancel(
+                _("Saved memory state will not be part of the snapshot"),
+                _("The domain is currently saved. Due to technical "
+                  "limitations that saved memory state will not become part "
+                  "of the snapshot. Running it later will be the same as "
+                  "having forced the system off mid-flight. It is "
+                  "recommended to snapshot either the running or shut down "
+                  "system instead."))
+            if not result:
+                return
         self._snapshot_new.show(self.topwin)
 
     def _on_refresh_clicked(self, ignore):
-- 
2.24.1


_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux