Bug 1427543 - A disk that is about to be deleted should be discarded first if it is attached to at least one vm with Pass Discard enabled
Summary: A disk that is about to be deleted should be discarded first if it is attache...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: future
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-4.1.2
: 4.1.2
Assignee: Idan Shaby
QA Contact: Kevin Alon Goldblatt
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-02-28 14:47 UTC by Idan Shaby
Modified: 2017-05-23 08:22 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Up until now, a disk would have been discarded only if its block storage domain's Discard After Delete was enabled. From now and on, a disk will be discarded also if it is attached to at least one virtual machine with Pass Discard enabled. The logic behind this is that if the user wanted "live" discarding, he will probably want to discard the whole disk when it is removed even if its storage domain's Discard After Delete property is disabled. For more information, please refer to the "Discard After Delete" feature page - http://www.ovirt.org/develop/release-management/features/storage/discard-after-delete/ .
Clone Of:
Environment:
Last Closed: 2017-05-23 08:22:33 UTC
oVirt Team: Storage
Embargoed:
amureini: ovirt-4.1?
amureini: exception?
amureini: planning_ack?
amureini: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 73163 0 master MERGED backend: generic type of PostDeleteActionHandler::fixParameters 2017-03-01 09:20:25 UTC
oVirt gerrit 73164 0 master MERGED backend: change Discard After Delete logic 2017-03-01 09:20:22 UTC
oVirt gerrit 73347 0 ovirt-engine-4.1 MERGED backend: generic type of PostDeleteActionHandler::fixParameters 2017-03-01 16:35:50 UTC
oVirt gerrit 73348 0 ovirt-engine-4.1 MERGED backend: change Discard After Delete logic 2017-03-01 16:35:54 UTC

Description Idan Shaby 2017-02-28 14:47:58 UTC
Description of problem:
Up until now, DAD (Discard After Delete) [1] was not related to Pass Discard [2], i.e a disk was first discarded before it was deleted only if its block storage domain's DAD value was true.
That created a situation where a vm disk that had its Pass Discard enabled was not discarded before it was deleted if its storage domain's DAD value was disabled. This situation does not make sense because if the user specifies that he wants "live" discarding (Pass Discard), he will most likely want to discard the whole disk before it's deleted.

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

How reproducible:
100%

Additional info:
[1] http://www.ovirt.org/develop/release-management/features/storage/discard-after-delete
[2] http://www.ovirt.org/develop/release-management/features/storage/pass-discard-from-guest-to-underlying-storage/

Comment 1 Yaniv Kaul 2017-02-28 15:44:39 UTC
Severity?

Comment 2 Allon Mureinik 2017-02-28 19:58:51 UTC
(In reply to Yaniv Kaul from comment #1)
> Severity?
It's a medium severity bug, IMHO.
It's not terrible (after all, we supported block storage but didn't discard deleted disks since 3.1), but it's ugly, and unintuitive - if I delete something from within the guest a discard would (eventually/potentially) be issued to the array, but if I delete the whole thing it wouldn't.
That doesn't make too much sense from a user's perspective.

Comment 5 Kevin Alon Goldblatt 2017-05-15 17:04:43 UTC
Verfied with the following code:
------------------------------------
ovirt-engine-4.1.2.1-0.1.el7.noarch
rhevm-4.1.2.1-0.1.el7.noarch
vdsm-4.19.14-1.el7ev.x86_64


Verfied with the following scenario:
--------------------------------------
1. Created 2 VMs and attache the same shared disk
2. Verfiied the scenario's of Wipe after Delete and Discard Enabled

Moving to verified!


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