Red Hat Bugzilla – Bug 1288366
NPE when moving a LUN disk in REST API
Last modified: 2016-03-10 07:04:52 EST
Is it possible to back-port to REHV 3.3?
+++ This bug was initially created as a clone of Bug #1126425 +++
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):
Steps to Reproduce:
1. Move a LUN disk, example:
There was always an NPE in this unreasonable scenario.
Both try to extract guid from imageId, which is null:
Guid imageId = asGuid(getDisk().getImageId()); //(BackendVmDiskResource)
Guid imageId = asGuid(get().getImageId()); //(BackendDiskResource X 2)
--- Additional comment from Allon Mureinik on 2014-08-27 13:00:53 EDT ---
The operation should fail in any event, just with a better error than NPE. Pushing out to 3.5.1
--- Additional comment from Allon Mureinik on 2014-09-18 06:21:22 EDT ---
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.
--- Additional comment from Idan Shaby on 2014-09-30 04:59:31 EDT ---
Tracking - Bug 1147860.
VM LUN disks:
Export - Bug 1147861.
Move - Bug 1147863.
Export - Bug 1147870
Move - Bug 1147873
Copy - Bug 1147874
--- Additional comment from Carlos Mestre González on 2015-09-22 06:59:28 EDT ---
[Cannot move Virtual Machine Disk. Action is unsupported for the specified disk type (LUN).]
--- Additional comment from Sandro Bonazzola on 2015-11-04 06:29:25 EST ---
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.
Setting this bug as VERIFIED, based on Carlos' work described in https://bugzilla.redhat.com/show_bug.cgi?id=1126425#c4.
When RHEV (as opposed to oVirt) 3.6 is released, this bug should be set to CLOSED CURRENTRELEASE.
RHEV 3.6.0 has been released, setting status to CLOSED CURRENTRELEASE