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

use_initialization in vm start #94

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gekorob
Copy link
Contributor

@gekorob gekorob commented Jul 10, 2020

Since the version 4.3.1 of the ovirt-engine-sdk-ruby is possible to start a vm passing the use_initialization param.
This can be an OS agnostic way to pass init data to the starting vm

depends on:
#93

@shiramax shiramax self-requested a review September 16, 2020 08:04
@shiramax shiramax self-assigned this Sep 16, 2020
@shiramax
Copy link
Contributor

@gekorob thanks for your contribution!

  1. could you please rebase?
  2. could you explain a bit about use_initialization and why you need it?
  3. Do you have another PR in foreman core?

Since the version 4.3.1 of the ovirt-engine-sdk-ruby is possible
to start a vm passing the use_initialization param.
In this way, according to the VM's OS type it'll be possible to set the
correct init section (cloudinit, sysprep, ignition)
@gekorob
Copy link
Contributor Author

gekorob commented Oct 13, 2020

@shiramax

  1. done
  2. it was in response to the problem reported in https://community.theforeman.org/t/foreman-windows-and-ovirt/19365
  3. no, I was working to manageiq-providers-ovirt and ovirt-engine-sdk-ruby in a very recent past.

Rubocop is failing for too many lines in the class, but I think it's a constraint difficult to satisfy in this case.
If you think this PR is not good to merge feel free to close it.

@shiramax
Copy link
Contributor

@gekorob I don't think that PR is not good for merging, but as of now no one is using this function start_with_initialization, so I wonder if another PR in foreman core is needed.

@shiramax shiramax removed their assignment May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants