-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
Assume raw format for image #7
Assume raw format for image #7
Conversation
8fc0f42
to
4523499
Compare
Is it a good idea to hardcode to the raw format over letting QEMU do probing? If someone has an image in a different format then it won't be able to boot if we hardcode to raw. Or is raw pretty much ubiquitous for Pi images? // CC @JamesReynolds |
@lukechilds I have never seen a qcow2 or vmdk image of a raspberry pi disk, I'm sure that raw is ubiquitous. However, adding ".img" is fairly ubiquitous too, so making ".img" default to raw could be a reasonable compromise? |
That's a good idea. @mrtwnklr are you able to check for the .img extension first? If there is an .img extension specify the raw format, if no .img extension just let QEMU do it's probing. |
Yes, that's a good idea. |
Oh of course, .img is hardcoded in the entrypoint. In that case I don't think the check is necessary. By requiring a .img file I guess we are implicitly saying we expect the raw format. |
@mrtwnklr are you able to just rebase this on top of the latest changes and then this is good to be merged. Thanks! |
abab529
to
7d0e25b
Compare
7d0e25b
to
e0ea1b6
Compare
Ok, that's it. |
Thanks @mrtwnklr! |
Assume raw format for image (lukechilds#7)
This change fixes the following warning when starting a container: