-
Notifications
You must be signed in to change notification settings - Fork 621
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
limactl depends on qemu or qemu-img, but lima brew package does not depend on qemu #3169
Comments
I don't think qemu should be depended on, as it is required only for rarely-used additional disks when vmType is set to vz. |
* Tested with env/regional-dr.yaml cluster creation on mac * Updated lima version in test/README.md * Updated test/README.md to install qemu for macOS due to lima dependency issue ([issue #3169](lima-vm/lima#3169)) Signed-off-by: Parikshith <[email protected]>
* Tested with env/regional-dr.yaml cluster creation on mac * Updated lima version in test/README.md * Updated test/README.md to install qemu for macOS due to lima dependency with qemu ([issue #3169](lima-vm/lima#3169)) Signed-off-by: Parikshith <[email protected]>
* Tested with env/regional-dr.yaml cluster creation on mac * Updated lima version in test/README.md * Updated test/README.md to install qemu for macOS due to lima dependency with qemu ([issue #3169](lima-vm/lima#3169)) Signed-off-by: Parikshith <[email protected]>
It makes sense, so we want a soft dependency. If qemu-image is available, we can support qcow2 format when creating or resizing images. If not, trying to create qcow2 format or resize qcow2 image can fail with helpful message like:
And same if you try to resize a qcow2 disk. To make this change we need to change the default format to be raw. This may be break some users, but using a default which is not available in all case is a mistake that we have to fix sooner or later. But if we remove the unneeded dependency on qemu-img for raw images (#2853), this issue is mostly resolved, since with vz there is no use case for creating or resizing disk in qcow2 format. |
The default format for what? For additional disks? I don't understand: if we can read qcow2 disks (to convert to raw format), why can't we also create empty/unformatted qcow2 disks? |
* Tested with env/regional-dr.yaml cluster creation on mac * Updated lima version in test/README.md * Updated test/README.md to install qemu for macOS due to lima dependency with qemu ([issue #3169](lima-vm/lima#3169)) Signed-off-by: Parikshith <[email protected]>
Description
When installing lima via brew, qemu is not installed.
limactl start
works with vz and create the root disk without qemu-img, but creating a vm with additional disks will fail when creating the disk:This failure could be avoided if we remove the unneeded dependency on qemu-img (#2853), but we still depend on qemu-img to create qcow2 disks. We also depend on qemu for the limactl snapshot command.
If we have a dependency we should depend on the package we depend on for best user experience.
If we want to introduce a soft dependency, that we can work with or without, we need to change the code so qcow2 format and qemu vmType are available only if qemu-img is available. The online help can recommend installing additional packages.
Possible fixes:
Related issues:
limactl disk
should not depend on qemu-img for raw disks #2853The text was updated successfully, but these errors were encountered: