Bug 1126425

Summary: NPE when moving a LUN disk in REST API
Product: [oVirt] ovirt-engine Reporter: Vered Volansky <vered>
Component: RestAPIAssignee: Vered Volansky <vered>
Status: CLOSED CURRENTRELEASE QA Contact: Carlos Mestre González <cmestreg>
Severity: low Docs Contact:
Priority: low    
Version: 3.4.4CC: amureini, bugs, ecohen, gklein, ishaby, rbalakri, tnisan, yeylon, ylavi
Target Milestone: ovirt-3.6.0-rcFlags: rule-engine: ovirt-3.6.0+
ylavi: planning_ack+
rule-engine: devel_ack+
rule-engine: testing_ack+
Target Release: 3.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1288366 (view as bug list) Environment:
Last Closed: 2015-11-04 11:29:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1147860, 1288366    

Description Vered Volansky 2014-08-04 12:14:55 UTC 
Description of problem:
When moving a LUN disk attached to a VM using REST API - getting NPE.
The same scenario with a floating disk should also result in an NPE, as well as copy disk.


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


How reproducible:
100%

Steps to Reproduce:
1. Move a LUN disk, example:
http://localhost:8080/api/vms/5b1ee280-4251-489a-899c-8e9dac73c327/disks/bdca3776-6597-4323-84f1-4ea75f7d269d/move
POST

Body:
<action>
    <storage_domain id="7158eae9-4886-4af7-af0b-d668ecee9e4f"/>
</action>
2. NPE

Actual results:
NPE

Expected results:
CDA message

Additional info:
There was always an NPE in this unreasonable scenario.
BackendVmDiskResource.move()
BackendDiskResource.move()
BackendDiskResource.copy()

Both try to extract guid from imageId, which is null:
Guid imageId = asGuid(getDisk().getImageId()); //(BackendVmDiskResource)
Guid imageId = asGuid(get().getImageId()); //(BackendDiskResource X 2)
Comment 1 Allon Mureinik 2014-08-27 17:00:53 UTC 
The operation should fail in any event, just with a better error than NPE. Pushing out to 3.5.1
Comment 2 Allon Mureinik 2014-09-18 10:21:22 UTC 
This requires moving around a LOT of code, for a small benefit.
Pushing out to 3.6.0.

Idan - please create a tracker and open bugs for the similar issues we have in other Disks actions.
Thanks.
Comment 3 Idan Shaby 2014-09-30 08:59:31 UTC 
Tracking - Bug 1147860.

VM LUN disks:
Export - Bug 1147861.
Move - Bug 1147863.

LUN disks:
Export - Bug 1147870
Move - Bug 1147873
Copy - Bug 1147874
Comment 4 Carlos Mestre González 2015-09-22 10:59:28 UTC 
<fault>
<reason>Operation Failed</reason>
<detail>
[Cannot move Virtual Machine Disk. Action is unsupported for the specified disk type (LUN).]
</detail>

version: 3.6.0-13
Comment 5 Sandro Bonazzola 2015-11-04 11:29:25 UTC 
oVirt 3.6.0 has been released on November 4th, 2015 and should fix this issue.
If problems still persist, please open a new BZ and reference this one.