Bug 1741104 - Create a specific signature for each flow in the 'Remove' REST-API interface for StorageDomainTemplate/Vm and AttachedStorageDomainDisk
Summary: Create a specific signature for each flow in the 'Remove' REST-API interface ...
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.3.2
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact: Avihai
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-08-14 09:35 UTC by Eyal Shenitzky
Modified: 2022-01-18 12:18 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-08-24 13:15:25 UTC
oVirt Team: Storage
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-36311 0 None None None 2021-08-24 13:17:19 UTC

Description Eyal Shenitzky 2019-08-14 09:35:01 UTC
Description of problem:

Currently, the path for removing an entity (VM / Template) is the same path as removing an entity which resides on an export domain.
So before removing the entity, we should decide whether that entity is an unregistered entity or not.

In order to decide whether the entity is unregistered or not a query sent to the backend, to avoid that query we should split the path for unregistered entities or have a specific signature for each flow in the 'Remove' interface


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Tal Nisan 2019-08-19 14:12:55 UTC
Eyal, changing the path is basically breaking the current API behavior, something we should not do, what do you suggest?

Comment 2 Eyal Shenitzky 2019-08-20 07:01:57 UTC
So I think it will be better to go on the second option to overload the 'remove' interface so it will distinct between unregistered entity remove and exported VM removal.

Comment 3 Eyal Shenitzky 2021-08-24 13:15:25 UTC
This bug/RFE is more than 2 years old and it didn't get enough attention so far, therefore it can be closed. 
Please re-open if this RFE is needed.


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