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

OSs that support MTV #583

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

Conversation

RichardHoch
Copy link
Collaborator

@RichardHoch RichardHoch commented Nov 11, 2024

MTV 2.7

Resolves https://issues.redhat.com/browse/MTV-1550 by linking to a better list of the operating systems that support MTV and by adding more information about the use of RHEL 8 vs. RHEL 9.

Preview: https://file.corp.redhat.com/rhoch/os_supporting_mtv/html-single/#source-vm-prerequisites_mtv

@RichardHoch RichardHoch force-pushed the os_supporting_mtv branch 2 times, most recently from 35c5a09 to 5fb4482 Compare November 11, 2024 13:38
@nunzy1
Copy link

nunzy1 commented Nov 11, 2024

Hi Richard. Minor comments. Feel free to accept or ignore. :-) I looked at other sections that seem to vary their pre-req's, but this is how I'd update this one.

  • ISO/CDROM disks are unmounted.
  • Each NIC contains one IPv4 and/or one IPv6 address.
  • The VM’s operating system is certified and supported as a guest operating system for conversions.

Note: You can check that the operating system is supported by referring to the table in this article that discusses VM conversions. (I would make the link within some portion of the sentence. The article name, imo, is too long and confusing, and mentions RHEL 7, which is on maintenance support.)

There’s no bullet for this one: RHEL 8 is used with MTV 2.6.z and RHEL 9 is used with MTV 2.7.z. Does this bullet mean that MTV 2.6.z requires RHEL 8 and MTV 2.7.z requires RHEL 9? If so

  • VMs running MTV 2.6.z are running RHEL 8.
  • VMs that are running MTV 2.6.z are running RHEL 9.
  • No VM names contain a period (.).

Note: Migration Toolkit for Virtualization (MTV) changes any period in a VM name to a dash (-).

  • No VM name duplicates any other VM name in the OpenShift Virtualization environment.

@RichardHoch
Copy link
Collaborator Author

@nunzy1 I made many of the changes you suggested. Could you please review the PR again?

@seanhaselden
Copy link

Looks good to me!

Copy link
Contributor

@fabiand fabiand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@RichardHoch RichardHoch force-pushed the os_supporting_mtv branch 2 times, most recently from 25ef48d to c07338e Compare December 12, 2024 09:51
@RichardHoch
Copy link
Collaborator Author

@anarnold97 I made changes based on your suggestions. Please review this PR again.

====

* VMs that you want to migrate with MTV 2.6._z_ run on RHEL 8.
* VMs that you want to migrate with MTV 2.7._z_ run on RHEL 9.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RichardHoch - thanks for fixing @nunzy1 error

@RichardHoch
Copy link
Collaborator Author

@anarnold97 This PR can be merged and published as soon as it passes all reviews.

* ISO/CDROM disks must be unmounted.
* Each NIC must contain one IPv4 and/or one IPv6 address.
* The operating system of a VM must be certified and supported as a link:https://access.redhat.com/articles/973163#ocpvirt[guest operating system with {virt}].
* ISO/CDROM disks are unmounted.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not saying this is wrong, but please can I double-check the format of ISO/CDROM

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anarnold97 Good call. Fixed.

Copy link
Collaborator

@anarnold97 anarnold97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but please check ISO/CDROM

Signed-off-by: RichardHoch <[email protected]>
Copy link
Collaborator

@anarnold97 anarnold97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

5 participants