-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[Feature]: Support Podman Machine provider selection using containers.conf #17116
Comments
This would be really useful I would also suggest the following
Update oh yes AppleHV is a framework not a complete VM solution it’s a framework so you would still need a full implementation of it such as Parallels. I suppose you could make one in swift but that’s probably a new project. |
@andrewcrook Thank you for your input! I suggest you to create a discussion in the dedicated section if there is no one already for these topics. This is because the issue reported here is only about very specific internal API extension and probably will be closed shortly as implemented and your suggestions will get lost. |
@baude @ashley-cui PTAL |
I think this is a good idea ... has the work been done in containers common already ? |
Change is merged into commons main branch containers/common@ead2e7d, but it was after the commons release has been cut for Podman release. I will prepare Podman part after commons will be updated. |
@rhatdan what say you? do you want to cut a common release or just vendor in a commit for now ? |
I also hope to finalize changes in this PR containers/common#1131 and get it into next release of commons. If just a bump is possible, then it is probably a more suitable option right now. |
Sure I would like to get this in. |
LookPath fix was merged into main in Common repo. Now need to wait for code pull or common release + code pull into Podman repo. |
A friendly reminder that this issue had no activity for 30 days. |
@arixmkii what is happening with this? Common has been updated, but it looks like your PR has stalled. |
Feature request description
There is already
[machine]
section in the containers.conf file and ReadCustomConfig method to read it. Extend this section to support optional provider record, with empty value representing default behavior (as it is now). Repacefunc GetSystemDefaultProvider() machine.Provider
withfunc GetSystemProvider(name string) machine.Provider
, where provider name must be passed by the caller and updates all the paths (some of them are already fetching the config, but others will need updates.This would be needed for support of WLS2 and QEMU on Windows, AppleHV and QEMU on Apple Silicon Macs.
Suggest potential solution
in the scope of #13006 the implementation would look like
This additionally allows to override the setting using environment variables for testing purposes. It also logs warning, when unexpected value is encountered.
Have you considered any alternatives?
Using it via command line arguments or environment variables only (no config), which would allow more flexibility. But from my experiments it has numerous downsides:
Additional context
This would encourage that normal usage scenario of a single machine provider per user session. But this matches the current behavior, where only a single provider per platform is supported.
I expected that this to be solved with implementation of AppleHV, but it looks like the implementation is lagging behind because of the lack of resources.
If there is another proposed implementation, which accomplishes the same result, then it is fine to withdraw this proposal and replace with what was already planned (I didn't manage to find it).
The text was updated successfully, but these errors were encountered: