After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 735687 - what happens when you delete current snapshot?
what happens when you delete current snapshot?
Status: RESOLVED INVALID
Product: gnome-boxes
Classification: Applications
Component: snapshots
3.13.x
Other Linux
: Normal normal
: 3.22
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-08-29 19:04 UTC by Zeeshan Ali
Modified: 2016-03-31 13:22 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Zeeshan Ali 2014-08-29 19:04:26 UTC
I deleted current snapshot successfully and another snapshot got chosen as current after wards. Is this whats supposed to happen? Maybe jimmac can give some input here.
Comment 1 Timm Bäder 2014-08-29 20:10:08 UTC
The one that was chosen was the one you've taken before the current one, i.e. the "most current" one (after deleting the current one... if that's clear). There's nothing we do here btw, it's just the behavior from libvirt and there isn't even code in Boxes to manually update that or anything, the draw function of SnapshotListRow simply uses is_current() to find out if it is the current snapshot and since we redraw the list when deleting a snapshot, one of them gets updated immediately and now indicates that it's the current one.
Comment 2 Lasse Schuirmann 2014-08-29 20:18:08 UTC
And IMO this behaviour makes totally sense. The current state is in every case a derivate of the most recent one (and not equal to the most recent one itself!) so the "current" snapshot is the parent of the current state. Maybe we could visualize that to make it more clear:

[ ] Snapshot 0001
 |
[ ] Snapshot 0002
 |
[+] Current state

In this example is_current would tell us true for Snapshot 0002 while we would show the user that the current state is the _derivative_ from this. This is more intuitive for me I think. (If we make visually clear that the current state is not a snapshot but you can click on the plus or so to make it one.)

jimmac?
Comment 3 Zeeshan Ali 2014-10-21 11:48:17 UTC
Closing..