Bug 1309693

Summary: [RFE] Enhance the functionality of add_disk for adv configs
Product: Red Hat CloudForms Management Engine Reporter: Josh Carter <jocarter>
Component: AutomateAssignee: Moti Asayag <masayag>
Status: CLOSED CURRENTRELEASE QA Contact: Milan Falešník <mfalesni>
Severity: high Docs Contact:
Priority: high    
Version: 5.5.0CC: cbolz, dajohnso, ealcaniz, fdewaley, gekis, gmccullo, jhardy, masayag, mfalesni, mkanoor, obarenbo, simaishi, tfitzger
Target Milestone: GAKeywords: FutureFeature, TestOnly
Target Release: 5.8.0   
Hardware: All   
OS: All   
Whiteboard: automate
Fixed In Version: 5.8.0.0 Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-06-12 16:08:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Comment 2 Greg McCullough 2016-02-25 14:45:46 UTC
*** Bug 1130060 has been marked as a duplicate of this bug. ***

Comment 3 Greg McCullough 2016-03-07 18:09:42 UTC
Related bugs:
Bug 1130060 - Support thin_provision flag.
Bug 1311221 - Support independent-persistent flags.

Comment 6 Edu Alcaniz 2016-11-16 14:33:29 UTC
We need an update about this RFE please?

Comment 7 Satoe Imaishi 2016-11-16 15:14:00 UTC
No update I can provide from the build side, as this hasn't been fixed yet. Based on the flags set on this bug, it's not targeted for cfme 5.7.

Please check with GregM for the status.

Comment 8 Edu Alcaniz 2016-11-17 22:31:47 UTC
Thanks for the update, Greg could you give more details please.

Comment 11 Edu Alcaniz 2017-01-24 11:28:53 UTC
David could you update the progress of the RFE please

Comment 12 Edu Alcaniz 2017-01-24 11:29:06 UTC
Dave could you update the progress of the RFE please

Comment 15 Greg McCullough 2017-02-02 14:15:13 UTC
Adding disks has been implemented in PR
https://github.com/ManageIQ/manageiq/pull/13318 for bug 1123068.

Comment 16 Satoe Imaishi 2017-02-02 16:17:58 UTC
bug 1414893 is the 5.7.z clone of bug 1123068

Comment 17 Edu Alcaniz 2017-03-06 09:55:34 UTC
(In reply to Greg McCullough from comment #15)
> Adding disks has been implemented in PR
> https://github.com/ManageIQ/manageiq/pull/13318 for bug 1123068.

This is for RHEV and customer use VMware, this RFE was created for VMware.

Comment 18 Greg McCullough 2017-03-06 13:32:54 UTC
Edu - VMware already supports these options.

Comment 21 Edu Alcaniz 2017-03-27 09:15:59 UTC
Could you give us an update please.

Comment 22 Milan Falešník 2017-03-27 09:49:50 UTC
Verified in 5.8.0.7 against RHEV 4.0

I executed this against a vm in automate:

vm.add_disk('testdisk2', 1536, :thin_provisioned => true)

and the disk got added as expected.

This feature throws a nil class exception in the log when used with RHEV 3.6 and the exception is not user-handleable. Should it either work or should it be posible to handle it?

Comment 23 Moti Asayag 2017-03-27 11:15:00 UTC
(In reply to Milan Falešník from comment #22)
> Verified in 5.8.0.7 against RHEV 4.0
> 
> I executed this against a vm in automate:
> 
> vm.add_disk('testdisk2', 1536, :thin_provisioned => true)
> 
> and the disk got added as expected.
> 
> This feature throws a nil class exception in the log when used with RHEV 3.6
> and the exception is not user-handleable. Should it either work or should it
> be possible to handle it?

RHEV 3.6 doesn't support disk attachments, therefore it shouldn't work for 3.6 hosts.

However, this issue should be tracked by Bug 1123068.
Could you comment there and attach the exception from log to that bug ?

Comment 24 Edu Alcaniz 2017-03-27 11:29:06 UTC
(In reply to Moti Asayag from comment #23)
> (In reply to Milan Falešník from comment #22)
> > Verified in 5.8.0.7 against RHEV 4.0
> > 
> > I executed this against a vm in automate:
> > 
> > vm.add_disk('testdisk2', 1536, :thin_provisioned => true)
> > 
> > and the disk got added as expected.
> > 
> > This feature throws a nil class exception in the log when used with RHEV 3.6
> > and the exception is not user-handleable. Should it either work or should it
> > be possible to handle it?
> 
> RHEV 3.6 doesn't support disk attachments, therefore it shouldn't work for
> 3.6 hosts.
> 
> However, this issue should be tracked by Bug 1123068.
> Could you comment there and attach the exception from log to that bug ?

Hi, my customer only works with VMware. if you need any test let me know.