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
Remove v3 api form SSA #394
Conversation
Remove the usage of the ovirt gem and version 3 api from the SSA Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535782
Checked commit borod108@de7eb0e with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 app/models/manageiq/providers/redhat/inventory/collector.rb
|
@agrare what say you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice @borod108 just the one spacing issue then this LGTM
@@ -151,7 +151,7 @@ def connect_to_ems(ost) | |||
$log.info "#{log_header}: Connecting to [#{ems_display_text}] for VM:[#{@vm_cfg_file}]" | |||
|
|||
begin | |||
ost.miqRhevm = ext_management_system.rhevm_inventory(:force_v3 => true) | |||
ost.miqRhevm = ManageIQ::Providers::Redhat::Inventory::Collector.new(ext_management_system, nil) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not for this, but I think we should make the Collector initializer initialize(manager, target = nil)
in core if it isn't required.
@borod108 these both say they depend on each other, if I had to guess I'd say this has to be merged first is that right? |
Remove v3 api form SSA (cherry picked from commit 343edb5) https://bugzilla.redhat.com/show_bug.cgi?id=1535782
Ivanchuk backport details:
|
Remove the usage of the ovirt gem and version 3 api from the SSA
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535782
Depends on: ManageIQ/manageiq-smartstate#98