-
Notifications
You must be signed in to change notification settings - Fork 118
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 check on number of runners when converting VM #395
Conversation
@miq-bot add-labels transformation, bug, gaprindashvili/yes, blocker |
ManageIQ::Automate::Transformation::TransformationHost::Common::VMTransform.new.main | ||
if == __FILE__ | ||
ManageIQ::Automate::Transformation::TransformationHost::Common::VMTransform.new.main | ||
end |
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.
@fdupont-redhat Please follow the changes from #390. You no longer need the if-block here.
result = Transformation::TransformationHosts::Common::Utils.remote_command(task, transformation_host, "/usr/bin/virt-v2v-wrapper.py", wrapper_options.to_json) | ||
raise result[:stderr] unless result[:rc].zero? | ||
@handle.log(:info, "Connecting to #{transformation_host.name} as #{transformation_host.authentication_userid}") if @debug | ||
@handle.log(:info, "Executing '/usr/bin/virt-v2v-wrapper.py'") |
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.
@fdupont-redhat
Do we have roles enabled for these tasks to be reliably run on a server where the code exists?
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.
Yes, the role is enabled as we filter out the hosts in AcquireTransformationHost.
Checked commits fabiendupont/manageiq-content@be2da65~...8310264 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
Remove check on number of runners when converting VM (cherry picked from commit 7a0d04d) https://bugzilla.redhat.com/show_bug.cgi?id=1610550
Gaprindashvili backport details:
|
When launching the conversion of the VM with virt-v2v, we check that the number of runners is not above the max number of runners per provider. However this check has already been done when acquiring the conversion host, and it leads to pausing the last tasks (number defined by number of generic workers).
This PR removes this check, allowing the balancing to fill the pool of conversions.
Associated RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1600152