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

ISO images by default are uploaded as Raw/Thin Provision. And if you … #963

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

Conversation

sermakov-orion
Copy link
Contributor

@sermakov-orion sermakov-orion commented Sep 25, 2024

ISO images by default are uploaded as Raw/Thin Provision. And if you later move such disk to a block storage (FC/iSCSI) then it will be converted to Cow/Thin Provision. And this might corrupt the disk. To avoid this issue we suggested to always upload ISO images as Raw/Preallocated. In this case no conversion needed while moving to a block storage. And disk works fine in both types (block and non-block) storages.

Are you the owner of the code you are sending in, or do you have permission of the owner?

y

Copy link
Member

@sandrobonazzola sandrobonazzola left a comment

Choose a reason for hiding this comment

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

This branch has conflicts that must be resolved

…later move such disk to a block storage (FC/iSCSI) then it will be converted to Cow/Thin Provision. And this might corrupt the disk. To avoid this issue we suggested to always upload ISO images as Raw/Preallocated. In this case no conversion needed while moving to a block storage. And disk works fine in both types (block and non-block) storages.

Signed-off-by: Stepan Ermakov <[email protected]>
@sermakov-orion
Copy link
Contributor Author

@sandrobonazzola
The conflicts were resolved. The branch is up to date now

@sandrobonazzola sandrobonazzola requested review from sandrobonazzola and removed request for sgratch October 4, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants