Bug 1124817 - It is possible to start multiple VM snapshot delete operations
Summary: It is possible to start multiple VM snapshot delete operations
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: virt-manager
Version: 21
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Cole Robinson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-07-30 11:52 UTC by Martin Kolman
Modified: 2014-09-20 23:40 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-09-20 23:40:48 UTC


Attachments (Terms of Use)

Description Martin Kolman 2014-07-30 11:52:03 UTC
Description of problem:
It is possible to start a delete operation for single snapshot multiple times.

Version-Release number of selected component (if applicable):


How reproducible:
always

Steps to Reproduce:
1. create/have at least one VM snapshot
2. try to delete the snapshot
3. repeat 2

Actual results:
It is possible to start multiple snapshot delete operations and eventually Virt Manager will return this traceback:

Error deleting snapshot 'pre post F21 mas rebuild update': Domain snapshot not found: no domain snapshot with matching name 'pre post F21 mas rebuild update'

Traceback (most recent call last):
  File "/usr/share/virt-manager/virtManager/asyncjob.py", line 91, in cb_wrapper
    callback(asyncjob, *args, **kwargs)
  File "/usr/share/virt-manager/virtManager/asyncjob.py", line 127, in tmpcb
    callback(*args, **kwargs)
  File "/usr/share/virt-manager/virtManager/domain.py", line 183, in delete
    self._backend.delete()
  File "/usr/lib64/python2.7/site-packages/libvirt.py", line 4702, in delete
    if ret == -1: raise libvirtError ('virDomainSnapshotDelete() failed')
libvirtError: Domain snapshot not found: no domain snapshot with matching name 'pre post F21 mas rebuild update'

Also the snapshot appears to be gone.

Expected results:
It is not possible to start multiple delete operations for a single VM snapshot.

Comment 1 Martin Kolman 2014-07-30 11:54:19 UTC
Looks like the traceback appears multiple times, based on how many delete operations you managed to start.

Comment 2 Cole Robinson 2014-09-20 23:40:48 UTC
I think this was fixed by this commit, which is also in F21. Since it's a minor feature request, just closing as CURRENTRELEASE for f21


commit f5b29c0416be5d05f2d684070d84c7600601907c
Author: Giuseppe Scrivano <gscrivan@redhat.com>
Date:   Thu Jul 31 14:28:54 2014 +0200

    snapshots: allow deleting multiple snapshots
    
    Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1124818
    
    Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>


Note You need to log in before you can comment on or make changes to this bug.