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

Move vmware provisioning dialogs to vmware repo #18614

Merged

Conversation

d-m-u
Copy link
Contributor

@d-m-u d-m-u commented Apr 2, 2019

I have RFE about adding thick_eager prov option in vmware dialogs. It requires changing the dialogs. As a first step, we should move the vmware prov dialogs to the vmware repo.

it's precursor to https://bugzilla.redhat.com/show_bug.cgi?id=1633867

Related:

ManageIQ/manageiq-providers-vmware#384

@d-m-u
Copy link
Contributor Author

d-m-u commented Apr 2, 2019

@miq-bot assign @tinaafitz
@miq-bot add_label cleanup

@d-m-u
Copy link
Contributor Author

d-m-u commented Apr 2, 2019

@miq-bot add_reviewer @gmcculloug

@miq-bot miq-bot requested a review from gmcculloug April 2, 2019 14:04
@d-m-u
Copy link
Contributor Author

d-m-u commented Apr 2, 2019

Movin' movin' movin'
Movin' movin' movin'
Movin' movin' movin'
Movin' movin' movin'
Rawhide

@d-m-u d-m-u force-pushed the moving_vmware_prov_dialogs_to_vmware_repo branch from 8a44037 to 0d3e4d4 Compare April 2, 2019 14:24
@miq-bot
Copy link
Member

miq-bot commented Apr 2, 2019

Checked commit d-m-u@0d3e4d4 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

Copy link
Member

@gmcculloug gmcculloug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@gmcculloug gmcculloug merged commit b55e06f into ManageIQ:master Apr 2, 2019
@agrare agrare added this to the Sprint 109 Ending Apr 15, 2019 milestone Apr 4, 2019
@d-m-u d-m-u deleted the moving_vmware_prov_dialogs_to_vmware_repo branch September 26, 2019 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants