Bug 1019626 - [engine-backend] A disk isn't activated automatically when attaching it to a vm
Summary: [engine-backend] A disk isn't activated automatically when attaching it to a vm
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 3.3.0
Hardware: x86_64
OS: Unspecified
unspecified
high
Target Milestone: ---
: 3.3.0
Assignee: Ayal Baron
QA Contact: Leonid Natapov
URL:
Whiteboard: storage
Depends On: 1015683 1029605
Blocks: 3.3snap4
TreeView+ depends on / blocked
 
Reported: 2013-10-16 07:35 UTC by Elad
Modified: 2016-02-10 20:27 UTC (History)
12 users (show)

Fixed In Version: is26
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
oVirt Team: Storage
Target Upstream Version:
Embargoed:
amureini: Triaged+


Attachments (Terms of Use)
screenshot (31.65 KB, image/png)
2013-10-16 07:35 UTC, Elad
no flags Details
vdsm+enigne logs (1.34 MB, application/x-gzip)
2013-10-17 14:39 UTC, Elad
no flags Details

Description Elad 2013-10-16 07:35:44 UTC
Created attachment 812791 [details]
screenshot

Description of problem:
When importing a new image from glance and attaching it to vm, the disk is not activated automatically like regular disk.

Version-Release number of selected component (if applicable):
rhevm-3.3.0-0.25.beta1.el6ev.noarch

How reproducible:
100%

Steps to Reproduce:
1. add new external provider to RHEVM (glance images)
2. import an image from glance to rhevm
3. create a new vm and attach the new imported disk

Actual results:
The new imported disk is not activated when attaching it to a vm.

Expected results:
The new imported disk should be activated when attaching it to a vm like a regular disk.

Additional info: screenshot

Comment 1 Allon Mureinik 2013-10-17 14:30:58 UTC
Please attach the logs (engine+vdsm)

Comment 2 Elad 2013-10-17 14:39:11 UTC
Created attachment 813359 [details]
vdsm+enigne logs

Comment 3 Daniel Erez 2013-11-12 13:47:06 UTC
Caused by lack of no-arg constructor in AttachDettachVmDiskParameters class
(GWT 2.5 serialization issues).
Already fixed by http://gerrit.ovirt.org/#/c/18908/

Comment 4 Allon Mureinik 2013-11-13 07:16:25 UTC
Daniel, Einav - bug 1029605 seems like it may take a while, and we want to enable QA to test Glance flows.

http://gerrit.ovirt.org/#/c/18908/ Seems quite straight forward, and a good idea to merge on the 3.3 branch regardless of the rest of the fix for 1029605.
Can we backport it and move THIS bug to MODIFIED?

Comment 5 Einav Cohen 2013-11-13 12:25:25 UTC
(In reply to Allon Mureinik from comment #4)
> Daniel, Einav - bug 1029605 seems like it may take a while, and we want to
> enable QA to test Glance flows.
> 
> http://gerrit.ovirt.org/#/c/18908/ Seems quite straight forward, and a good
> idea to merge on the 3.3 branch regardless of the rest of the fix for
> 1029605.
> Can we backport it and move THIS bug to MODIFIED?

Hi Alon, we intend to backport all 3 patches associated with bug 1029605 to ovirt-engine-3.3, but I wanted to do backport all of them together, once all are merged to 'master' (although we indeed don't have to wait for that in order to backport the specific one that you mentioned).

while we are at it: any chance for a re-review on http://gerrit.ovirt.org/#/c/20274/?

Comment 6 Allon Mureinik 2013-11-27 09:55:47 UTC
Moved to MODIFIED since bug 1029605 is now MODIFIED too, in order for QA to validate once there's a build.

Comment 7 Ayal Baron 2013-11-27 10:40:57 UTC
(In reply to Allon Mureinik from comment #6)
> Moved to MODIFIED since bug 1029605 is now MODIFIED too, in order for QA to
> validate once there's a build.

Just putting on ON_QA with fixed in version - is26 (next build which will include the relevant patches)

Comment 8 Elad 2013-12-15 08:37:24 UTC
A disk which was originally imported as a glance image, is being activated automatically when attaching it to VM.

Verified using is26

Comment 9 Elad 2013-12-15 08:37:54 UTC
A disk which was originally imported as a glance image, is being activated automatically when attaching it to VM.

Verified using is26

Comment 10 Itamar Heim 2014-01-21 22:21:51 UTC
Closing - RHEV 3.3 Released

Comment 11 Itamar Heim 2014-01-21 22:27:07 UTC
Closing - RHEV 3.3 Released


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