-
Notifications
You must be signed in to change notification settings - Fork 24
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
MTV-1433 Rearranging prerequisites in the MTV user guide #591
base: main
Are you sure you want to change the base?
Conversation
cb6ca0b
to
f899fce
Compare
1cc1857
to
9160c8a
Compare
The configured file system overhead has a default value of 10%. If that is too low, the disk transfer fails due to lack of space. In this case, increase the file system overhead. | ||
|
||
If you need to reduce storage consumption, decrease the file system overhead. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nunzy1 -- I changed lines 17-19 just now. Please review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sentences are all good. :-) If the file system fails, they increase the overhead, as mentioned. The next sentence says that "if you need to reduce storage consumption," decrease the overhead. But, if I'm reading this right, if I decrease the overhead to reduce storage consumption, I risk the disk transfer failing. Do we tell them anywhere the scenario where they would decrease the overhead but not have the disk transfer fail?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally would not recommend reducing that overhead as it could cause failed migrations, so I think we can remove that sentence. But I would highlight that this applies only for the volumes on the PVC with volumeMode Filesystem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mnecas Changed.
@nunzy1 I made many of the changes you suggested and did a little other rewriting. Please review this PR again. Thanks. |
637ed1a
to
d01e70d
Compare
MTV 2.8
Partially resolves https://issues.redhat.com/browse/MTV-1433 by grouping prerequisites by vendor (VMware, RHV, etc.) where possible.
Previews:
Note -- Some port information is missing. This is indicated by "TBD".