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

The openshift source provider is missing from the list of providers mentioned in the CL migrating vms section #446

Open
sgratch opened this issue Nov 15, 2023 · 0 comments

Comments

@sgratch
Copy link
Contributor

sgratch commented Nov 15, 2023

Describe the bug

Under this doc section https://access.redhat.com/documentation/en-us/migration_toolkit_for_virtualization/2.5/html-single/installing_and_using_the_migration_toolkit_for_virtualization/index#migrating-virtual-machines-cli_mtv, in the procedure section, step no. 1.2, (Secret manifest )the following text:
Specify the type of source provider. Allowed values are ovirt, vsphere, openstack, and ova. This label is needed to verify the credentials are correct when the remote system is accessible and, for RHV, to retrieve the Manager CA certificate when a third-party certificate is specified.

Should be rephrased with the following:
Specify the type of source provider. Allowed values are ovirt, vsphere, openstack, **openshift** and ova. This label is needed to verify the credentials are correct when the remote system is accessible and, for RHV, to retrieve the Manager CA certificate when a third-party certificate is specified.

The fix includes adding openshift as a source provider.

We added supporting OpenShift as a source provider in 2.5, so this fix is relevant for 2.5 version.

Notes:

  1. This same text is also displayed in the console plugin UI: https://github.com/kubev2v/forklift-console-plugin/blob/7ac736db94a33c1efe4ce26e6e[…]-console-plugin/locales/en/plugin__forklift-console-plugin.json

  2. Going over this whole CLI doc section, it seems that openshift (and ova) as source providers are missing in other places where type is mentioned (e.g Provider manifest ), so maybe worth adding it as well.

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

No branches or pull requests

1 participant