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

Extract raw_* vm operations to providers #19452

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Oct 31, 2019

Instead of having all of the raw_* operations defined in core calling
run_command_via_parent, leave the raw_ operations undefined and allow
them to be implemented in the different providers.

Cross Repo Tests: ManageIQ/manageiq-cross_repo-tests#6
Pluggability Issue: #19440

Independent: (can be merged before or after this)

Depends: (these should be merged before this PR)

@agrare agrare force-pushed the extract_raw_vm_operations_to_providers branch from 3d1fbdc to 82a7549 Compare October 31, 2019 19:13
@miq-bot miq-bot added the wip label Oct 31, 2019
@agrare agrare force-pushed the extract_raw_vm_operations_to_providers branch 9 times, most recently from 6b013b2 to d293f6b Compare November 4, 2019 14:10
@agrare agrare changed the title [WIP] Extract raw_* vm operations to providers Extract raw_* vm operations to providers Nov 4, 2019
@miq-bot miq-bot removed the wip label Nov 4, 2019
@agrare
Copy link
Member Author

agrare commented Nov 7, 2019

@Fryguy can you take a look?

@agrare agrare force-pushed the extract_raw_vm_operations_to_providers branch from d293f6b to 6e492c1 Compare November 11, 2019 17:20
Instead of having all of the raw_* operations defined in core calling
run_command_via_parent, leave the raw_ operations undefined and allow
them to be implemented in the different providers.
@miq-bot
Copy link
Member

miq-bot commented Nov 11, 2019

Checked commit agrare@6e492c1 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
9 files checked, 0 offenses detected
Everything looks fine. ⭐

@Fryguy Fryguy merged commit 4551d7c into ManageIQ:master Nov 12, 2019
@Fryguy Fryguy added this to the Sprint 125 Ending Nov 25, 2019 milestone Nov 12, 2019
@agrare agrare deleted the extract_raw_vm_operations_to_providers branch November 12, 2019 21:59
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.

3 participants