-
Notifications
You must be signed in to change notification settings - Fork 897
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
Add conversion_host option to the SupportsFeatureMixin module #18232
Conversation
@djberg96 is the idea to check if it is possible to enable a resource to be a conversion host, or to check if one is a conversion host? |
@agrare The idea is to check if it's possible to enable a resource to be a conversion host. To see if a resource is a conversion host we can check to see if there's an existing relationship to the conversion_host table. That, or perhaps there's some other attribute on the v2v side, can't remember right now. |
@djberg96 yeah okay good, since we already have has_one conversion_host on vms and hosts 👍 |
Checked commits https://github.com/djberg96/manageiq/compare/9e069e08bd582cbc5885f226bb2e63c4ac4b3d81~...3e2a6fbf9ebb9fb09083b1a6d25d71e275e9c464 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@miq-bot add_label hammer/yes |
Add conversion_host option to the SupportsFeatureMixin module (cherry picked from commit c9d1e4d) https://bugzilla.redhat.com/show_bug.cgi?id=1694229
Hammer backport details:
|
…ost" This reverts commit 48dbd21.
Reverted the backport:
|
Add conversion_host option to the SupportsFeatureMixin module (cherry picked from commit c9d1e4d) https://bugzilla.redhat.com/show_bug.cgi?id=1702019
Hammer backport details:
|
This PR adds the
conversion_host
option to theSupportsFeature
mixin for conversion host support (or not).The idea here is that we can then use this for V2V, where individual resources (VM or possibly Host) within each provider repo can put this within their respective vm.rb or host.rb file(s). This will help us govern behavior elsewhere within the codebase via
supports :conversion_host
orsupports_not :conversion_host
.With this in place, the ManageIQ REST API can also take advantage of it in order to check for support. See here for an example of where this is already used:
https://github.com/ManageIQ/manageiq-api/blob/master/app/controllers/api/subcollections/security_groups.rb#L12
https://bugzilla.redhat.com/show_bug.cgi?id=1622728https://bugzilla.redhat.com/show_bug.cgi?id=1695810