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

Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) #2494

Merged

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Oct 23, 2017

Allow the user to enter a new disk controller type when adding a new disk to a VMWare VM via reconfigure

Links

https://www.pivotaltracker.com/story/show/152116440

Depends on ManageIQ/manageiq-providers-vmware#126

Before:
screenshot from 2017-10-26 11-15-56

After - reconfigure VM and Edit Existing Reconfigure Request:

screenshot from 2017-10-26 11-12-56
screenshot from 2017-10-26 11-14-03

@lgalis lgalis changed the title [WIP] Allow the user to enter a new disk controller type when adding a VMWare disk via reconfigure [WIP] Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 23, 2017
@miq-bot miq-bot added the wip label Oct 23, 2017
@lgalis lgalis force-pushed the add_disk_controller_to_vm_reconfigure branch from 8e11ada to c64297a Compare October 25, 2017 14:04
@lgalis lgalis changed the title [WIP] Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 25, 2017
@lgalis lgalis changed the title Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) [WIP\Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 25, 2017
@lgalis lgalis changed the title [WIP\Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 25, 2017
@miq-bot miq-bot removed the wip label Oct 25, 2017
@lgalis lgalis changed the title Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) [WIP] Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 25, 2017
@miq-bot miq-bot added the wip label Oct 25, 2017
@lgalis lgalis force-pushed the add_disk_controller_to_vm_reconfigure branch from 993d367 to 4abb9eb Compare October 25, 2017 20:18
@lgalis lgalis force-pushed the add_disk_controller_to_vm_reconfigure branch 2 times, most recently from e2b333f to acc3744 Compare October 26, 2017 13:20
@lgalis lgalis force-pushed the add_disk_controller_to_vm_reconfigure branch from acc3744 to 4ff43b2 Compare October 26, 2017 14:24
@miq-bot
Copy link
Member

miq-bot commented Oct 26, 2017

Checked commits lgalis/manageiq-ui-classic@10a49a0~...4ff43b2 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@lgalis lgalis changed the title [WIP] Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Allow the type of controller to be passed in when adding a disk to a Vm ( VMWare ) Oct 26, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 26, 2017

@lgalis Cannot apply the following label because they are not recognized: enahancement

@lgalis
Copy link
Contributor Author

lgalis commented Oct 26, 2017

@miq_bot add_label pending_core

@lgalis
Copy link
Contributor Author

lgalis commented Oct 26, 2017

@miq-bot add_label enhancement

@lgalis
Copy link
Contributor Author

lgalis commented Oct 26, 2017

@h-kataria, @lfu - please review

@miq-bot miq-bot removed the wip label Oct 26, 2017
@lgalis
Copy link
Contributor Author

lgalis commented Oct 26, 2017

@miq-bot add_label pending core

@agrare
Copy link
Member

agrare commented Oct 26, 2017

@lgalis the dependent PR has been merged
@miq-bot remove-label pending core

@lfu
Copy link
Member

lfu commented Oct 26, 2017

I got this when testing it.
controller type

@lgalis
Copy link
Contributor Author

lgalis commented Oct 26, 2017

@lfu - you need the updates from this PR: ManageIQ/manageiq-providers-vmware#126

@lfu
Copy link
Member

lfu commented Oct 26, 2017

@lgalis 👍 Updated my environment and the testing looked good! A new SCSI controller with specified type was created when it ran out of port on the existing controller.

@h-kataria h-kataria self-assigned this Oct 26, 2017
@h-kataria h-kataria added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 26, 2017
@h-kataria h-kataria merged commit 07823c6 into ManageIQ:master Oct 26, 2017
@lgalis lgalis deleted the add_disk_controller_to_vm_reconfigure branch October 26, 2017 20:38
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