Describe the bug

the link on the README is broken to the documentation site https://forklift-docs.konveyor.io/

Solution

not sure how this is hosted or updated

Screenshots

Describe the bug

Podman command for preview build no longer works because of a new Git security feature. The podman container fails with a "fatal: unsafe repository" error and the workaround in the error message does not appear to work.

Solution

Provide instructions for installing Asciidoctor, running "$ asciidoctor master.adoc" in the document directory, and then opening the master.html file in a browser. Delete the master.html file so that it is not included in a PR.

It would also be a good idea to add "master.html" to the gitignore file.

When creating a new plan is failed because of following error:

The new plan still can be shown on the plans page, migration can be started, VM could be imported successfully.

If creating a new plan is failed, the plan shouldn't be shown on the plans page.

Steps to reproduce:

1. Create a non cluster admin user

2. Create a role with following rules:
   
   Note that there is no 'delete' verb on networkmaps

3. Bind the user with the role

4. Log into cluster with the user, and create a new plan

Describe the bug

A clear and concise description of the bug. If there is an error in the published documentation, include the section number and the release version or provide a link to the section.

https://github.com/kubev2v/forklift-documentation#building-a-document-preview

`Create .jekyll-cache and _site directories:
for i in .jekyll-cache _site; do mkdir ${i} && chmod 777 ${i}; done`

Solution

Remove .jekyll-cache folder from repo or add command to check and delete it prior to creating the folder

A clear and concise description of the solution. Does it apply to older releases or only to the current release?

Screenshots

If applicable, add screenshots to help explain your problem.

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.

When accessing the documentation via web: https://forklift-docs.konveyor.io/documentation/doc-Migration_Toolkit_for_Virtualization/master/index.html

... Images do not load. For example this diagram:

Which is pointing to this URL:
https://forklift-docs.konveyor.io/documentation/doc-Migration_Toolkit_for_Virtualization/master/modules/images/136_Upstream_Migration_Toolkit_0121_mtv-workflow.svg
... that returns a 404