-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[Bug]: docker vscode extension v1.23.0 breaks podman compatibility #17306
Comments
If it works with the extension version In any case if there is something we are doing wrong we need the exact podman command and what the output is and what it should be in order to fix it. |
@Luap99 it's not the vscode's Docker extension's scope to maintain compatibility with podman, it works just fine with Docker. Podman advertises docker compatability, that's why I filed it here. My guess is that something changed in that version that is looking for something in the json output that podman isn't providing, I'll see if I can hunt down a specific commit. I could not get any specific output from the extension itself's output window, I'll see if the docker extension has some sort of trace log for the commands its running. |
@baude a blog on how to debug podman-machine would be nice (e.g., how to get REST API calls from the machine) |
Matching issue in vscode docker repo: microsoft/vscode-docker#3241 Comment explaining the change (they went to all-cli in 1.23): microsoft/vscode-docker#3766 (comment) |
FYI, with the 4.4 Podman release I see that it returns output that is more consistent with the Docker CLI, i.e line delimited JSON objects as the VS Code extension expects. The issue is that the casing of keys is different, e.g |
If we output with Id and ID would that work? |
@bwateratmsft mentioned in another issue that they're looking into Podman support, so it makes sense to loop them in before changing anything IMO. Edit: I see this has been discussed here, so all good I think? |
A friendly reminder that this issue had no activity for 30 days. |
facing same issue but notice that also podman dashboard is complaining about the docker socket compatibility. certainly revert back to docker vscode extension helps to make vscode to work again with podman, why podman dashboard is generating that warning. I followed the tutorial https://podman-desktop.io/docs/troubleshooting#warning-about-docker-compatibility-mode
but seems not emulating it. |
Probably caused by removing the |
Setting the DOCKER_HOST environment doesn't work? Have you reported this issue to the people who ship the docker vscode extension? (Microsoft?) or ask them to have a Podman extension? |
@rhatdan yes, it's been reported: microsoft/vscode-docker#3241 currently the only option is to downgrade the Docker extension. A quick fix is possible per #17306 (comment) but the Docker extension team has opted to instead provide full explicit support for podman (no ETA given so far) |
It's a pity that this problem is still not solved and we have to use an outdated version of the plugin. |
@n1hility PTAL |
For your information, a new extension for accessing podman from VS Code has come up by @dreamcatcher45 named Pod Manager. It is working fine with podman containers. Give it a try. https://marketplace.visualstudio.com/items?itemName=dreamcatcher45.podmanager&ssr=false#overview |
Very nice. |
Issue Description
Running
v1.22.2
, everything works fine. Inv1.23
+, it appears the json schema may have changed and it is expecting something podman doesn't provide, and errors with invalid json schema in the windows. Reverting tov1.22.2
fixes the issueSteps to reproduce the issue
Steps to reproduce the issue
Describe the results you received
Describe the results you expected
normal operation
podman info output
Podman in a container
No
Privileged Or Rootless
Rootless
Upstream Latest Release
Yes
Additional environment details
Additional environment details
Additional information
Possible dupe of #17229 and supposedly fixed by #16446 but I tried the 4.4 dev build and it didn't fix the issue.
The text was updated successfully, but these errors were encountered: