Bug 1445874 - When adding Disk with reconfiguration on vmware, after 16th Disk, a new controller is created hardcoded to Parallel Type
Summary: When adding Disk with reconfiguration on vmware, after 16th Disk, a new contr...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.9.0
Assignee: Adam Grare
QA Contact: Niyaz Akhtar Ansari
URL:
Whiteboard: vm:reconfigure:vmware
Depends On:
Blocks: 1452172 1459262
TreeView+ depends on / blocked
 
Reported: 2017-04-26 17:01 UTC by Loic Avenel
Modified: 2018-03-06 15:28 UTC (History)
7 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1452172 1459262 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:28:17 UTC
Category: ---
Cloudforms Team: VMware
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Loic Avenel 2017-04-26 17:01:13 UTC
Description of problem: When adding Disk with reconfiguration, after 16th Disk, a new controller is created hardcoded to Parallel Type


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


How reproducible: Always


Steps to Reproduce:
1. Add 15 disks to an existing VM with Controller type set to SAS
2. look at the 16th Disk Controller Type

Actual results: Storage Controller type is parallel


Expected results: Should be SAS like exiting Controller 


Additional info: This issue is reported by Accenture for Renault

Comment 4 CFME Bot 2017-05-04 18:22:57 UTC
New commit detected on ManageIQ/manageiq-gems-pending/master:
https://github.com/ManageIQ/manageiq-gems-pending/commit/8bec31972f46d9c26cba68e9d6b3b553eabde609

commit 8bec31972f46d9c26cba68e9d6b3b553eabde609
Author:     Adam Grare <agrare>
AuthorDate: Tue May 2 13:13:25 2017 -0400
Commit:     Adam Grare <agrare>
CommitDate: Tue May 2 13:25:23 2017 -0400

    Add method to get vm hardware and scsi controllers
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1445874

 lib/gems/pending/VMwareWebService/MiqVimVm.rb | 9 +++++++++
 1 file changed, 9 insertions(+)

Comment 5 CFME Bot 2017-05-04 18:23:02 UTC
New commit detected on ManageIQ/manageiq-gems-pending/master:
https://github.com/ManageIQ/manageiq-gems-pending/commit/7e13304de16fd5cc2c785a6f668f3585293b4303

commit 7e13304de16fd5cc2c785a6f668f3585293b4303
Author:     Adam Grare <agrare>
AuthorDate: Tue May 2 13:29:59 2017 -0400
Commit:     Adam Grare <agrare>
CommitDate: Wed May 3 12:19:52 2017 -0400

    Allow hardware to be passed to save a getProp call
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1445874

 lib/gems/pending/VMwareWebService/MiqVimVm.rb | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Comment 6 CFME Bot 2017-05-17 17:04:10 UTC
New commit detected on ManageIQ/manageiq-providers-vmware/master:
https://github.com/ManageIQ/manageiq-providers-vmware/commit/8acb41868b1e012ea27f12373468589392279e48

commit 8acb41868b1e012ea27f12373468589392279e48
Author:     Adam Grare <agrare>
AuthorDate: Tue May 2 13:15:49 2017 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu May 4 14:44:25 2017 -0400

    Make vm reconfig remove disks consistent with add
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1445874

 .../manageiq/providers/vmware/infra_manager/vm/reconfigure.rb  | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

Comment 7 CFME Bot 2017-05-17 17:04:15 UTC
New commit detected on ManageIQ/manageiq-providers-vmware/master:
https://github.com/ManageIQ/manageiq-providers-vmware/commit/529de224ce0ba00082aad294c7b7733f11ae3b3e

commit 529de224ce0ba00082aad294c7b7733f11ae3b3e
Author:     Adam Grare <agrare>
AuthorDate: Tue May 2 15:00:56 2017 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu May 4 14:44:25 2017 -0400

    Use hardware to find last scsi controller type
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1445874

 .../vmware/infra_manager/vm/reconfigure.rb         | 37 +++++++++++++++-------
 1 file changed, 25 insertions(+), 12 deletions(-)

Comment 10 Niyaz Akhtar Ansari 2017-12-13 05:25:33 UTC
Verified in 5.9.0.11


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