Skip to content

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

@gekorob gekorob force-pushed the using_initialization branch from 664913d to 55f36d6 Compare July 13, 2020 12:06
@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 gekorob force-pushed the using_initialization branch from 55f36d6 to ea58965 Compare October 13, 2020 13:27
@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