Bug 858618 - RSDL: missing method for removing VM from storage domain
Summary: RSDL: missing method for removing VM from storage domain
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-restapi
Version: 3.1.0
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: ---
: 3.1.0
Assignee: Michael Pasternak
QA Contact: Jakub Libosvar
URL:
Whiteboard: storage
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-09-19 08:54 UTC by Jakub Libosvar
Modified: 2016-02-10 17:01 UTC (History)
12 users (show)

Fixed In Version: si20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-04 20:00:53 UTC
oVirt Team: Storage
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jakub Libosvar 2012-09-19 08:54:11 UTC
Description of problem:
In /api?rsdl is missing delete method for removing vm from given storage domain. I assume it leads to missing delete() method on instance of StorageDomainVM class in ovirtsdk.

Version-Release number of selected component (if applicable):
rhevm-restapi-3.1.0-16.el6ev.noarch

How reproducible:
Always

Steps to Reproduce:
1. Take a look to api?rsdl and search for /api/storagedomains/{storagedomain:id}/vms/{vm:id} rel="delete"
or
1. try to remove vm from storage domain using ovirtsdk
  
Actual results:
(Pdb) type(vmObj)
<class 'ovirtsdk.infrastructure.brokers.StorageDomainVM'>
(Pdb) vmObj.delete()
*** AttributeError: 'VM' object has no attribute 'delete'

Comment 1 Ayal Baron 2012-09-19 09:19:24 UTC
This was deprecated in 3.1 seeing as a VM is no longer limited to a specific storage domain.

You need to go through /api/vms/...

Michael, why do we even have 'vms' under storage domains?

Comment 2 Michael Pasternak 2012-09-19 09:27:21 UTC
(In reply to comment #1)
> This was deprecated in 3.1 seeing as a VM is no longer limited to a specific
> storage domain.

jakub refers to export domain ...

> 
> You need to go through /api/vms/.
> 
> Michael, why do we even have 'vms' under storage domains?

cause otherwise you won't be able to import it (vms shown only under export domain)

Comment 3 Michael Pasternak 2012-10-02 20:50:59 UTC
http://gerrit.ovirt.org/#/c/8318/

Comment 4 Michael Pasternak 2012-10-03 10:02:47 UTC
https://gerrit.eng.lab.tlv.redhat.com/2433

Comment 5 Michael Pasternak 2012-10-03 10:47:25 UTC
same was true for template

Comment 7 Petr Dufek 2012-10-15 13:31:29 UTC
verified in si20


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