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

Automate - Retirement - Change Infrastructure for Microsoft SCVMM #6358

Merged

Conversation

billfitzgerald0120
Copy link
Contributor

vendor to call generic and check_pre_retirement methods.

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

@billfitzgerald0120
Copy link
Contributor Author

@mkanoor
Please review

@miq-bot
Copy link
Member

miq-bot commented Jan 26, 2016

Checked commit billfitzgerald0120@b4ceb77 with ruby 2.2.3, rubocop 0.34.2, and haml-lint 0.13.0
0 files checked, 0 offenses detected
Everything looks good. ⭐

@mkanoor
Copy link
Contributor

mkanoor commented Jan 28, 2016

👍

@billfitzgerald0120
Copy link
Contributor Author

@gmcculloug
Please review

@gmcculloug
Copy link
Member

Just to be clear these are changes to support Microsoft SCVMM only (not Azure).

@gmcculloug gmcculloug changed the title Automate - Retirement - Change Infrastructure for Microsoft Automate - Retirement - Change Infrastructure for Microsoft SCVMM Jan 29, 2016
gmcculloug added a commit that referenced this pull request Jan 29, 2016
…icrosoft

Automate - Retirement - Change Infrastructure for Microsoft SCVMM
@gmcculloug gmcculloug merged commit a3f4347 into ManageIQ:master Jan 29, 2016
@gmcculloug gmcculloug added this to the Sprint 36 Ending Feb 16, 2016 milestone Jan 29, 2016
durandom pushed a commit to durandom/manageiq that referenced this pull request Feb 29, 2016
Change Infrastructure retirement for Microsoft vendor to call generic

pre_retirement and check_pre_retirement methods.

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

Clean cherry pick

original BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1297351

original PR: ManageIQ#6358

See merge request !783
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.

None yet

5 participants