Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix VM Reconfigure error with more than 1 disk on the VM #8791

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented May 18, 2016

When calculating the controller key and unit number detect was
used so that if there were more than 1 scsi controller or
virtual disk on the target vm the new disk would be added to
a populated slot on the scsi controller, leading to the error
message from VMware "Invalid configuration for device '0'"

https://bugzilla.redhat.com/show_bug.cgi?id=1336812

When calculating the controller key and unit number detect was
used so that if there were more than 1 scsi controller or
virtual disk on the target vm the new disk would be added to
a populated slot on the scsi controller, leading to the error
message from VMware "Invalid configuration for device '0'"

https://bugzilla.redhat.com/show_bug.cgi?id=1336812
@agrare agrare force-pushed the bz_1336812_vm_reconfig_more_than_one_disk branch from 0cd4e02 to 79ae05b Compare May 18, 2016 18:20
@miq-bot
Copy link
Member

miq-bot commented May 18, 2016

Checked commit agrare@79ae05b with ruby 2.2.3, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 🍰

@blomquisg blomquisg merged commit ac09c6b into ManageIQ:master May 19, 2016
@blomquisg blomquisg added this to the Sprint 41 Ending May 30, 2016 milestone May 19, 2016
@agrare agrare deleted the bz_1336812_vm_reconfig_more_than_one_disk branch May 19, 2016 12:29
chessbyte pushed a commit that referenced this pull request May 19, 2016
…_one_disk

Fix VM Reconfigure error with more than 1 disk on the VM
(cherry picked from commit ac09c6b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants