Skip to content

Conversation

@RichardHoch
Copy link
Collaborator

@RichardHoch RichardHoch commented Nov 20, 2024

@RichardHoch RichardHoch changed the title WIP Rearranging prerequisites in the MTV user guide MTVC-1433 WIP Rearranging prerequisites in the MTV user guide Nov 25, 2024
@RichardHoch RichardHoch changed the title MTVC-1433 WIP Rearranging prerequisites in the MTV user guide MTV-1433 Rearranging prerequisites in the MTV user guide Nov 27, 2024
@RichardHoch RichardHoch marked this pull request as ready for review November 28, 2024 11:52
@RichardHoch
Copy link
Collaborator Author

@nunzy1 I made many of the changes you suggested and did a little other rewriting. Please review this PR again. Thanks.

@RichardHoch
Copy link
Collaborator Author

@mnecas @fabiand Please review this PR. It is the 4th and final PR (hopefully) in the "restructure the user guide" epic.

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.

A few things to change.


[TIP]
====
It is strongly recommended to disable hibernation because {project-first} does not support migrating hibernated VMs.
Copy link
Contributor

Choose a reason for hiding this comment

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

If we know for sure that it fails, then we should reword to

Hibernation must be disabled because {project-first} does not support migrating hibernated VMs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mnecas Do we know for sure migration fails for hibernated VMs? If I remember correctly, it might fail, it might not, so we phrased it as "strongly recommended to disable", but please keep me honest. @fabiand

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@fabiand @mnecas Please note that information about ports is still missing: https://issues.redhat.com/browse/MTV-1714

@RichardHoch RichardHoch requested a review from fabiand January 12, 2025 14:15
@RichardHoch RichardHoch force-pushed the prereqs_by_provider branch 2 times, most recently from 1f61667 to 975217b Compare January 15, 2025 09:25
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