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

Does not work with podman #487

Closed
td0m opened this issue Mar 9, 2023 · 2 comments · Fixed by #494
Closed

Does not work with podman #487

td0m opened this issue Mar 9, 2023 · 2 comments · Fixed by #494
Labels
triage Investigation required

Comments

@td0m
Copy link

td0m commented Mar 9, 2023

Expected Behaviour
Setting DOCKER_HOST to podman socket and disabling RYUK should work correctly such as with other testcontainer implementations (works fine with go and java)

Actual Behaviour
It fails on trying to obtain credentials here because the docker related binaries are not there..

Testcontainer Logs

Error: An error occurred listing credentials

Steps to Reproduce
Start any generic container after disabling RYUK and setting DOCKER_HOST to a podman socket.

Environment Information

  • Operating System: MacOS
  • Docker Version: -
  • Node version: LTS
  • Testcontainers version: 9.1.3
@cristianrgreco
Copy link
Collaborator

Hi @td0m, the implementation is the same as tc-java (https://github.com/testcontainers/testcontainers-java/blob/f591ac7ef68119503473bd7d09b7b3a5cf4b3db8/core/src/main/java/org/testcontainers/utility/RegistryAuthLocator.java#L332), could you please elaborate on why that's causing an issue?

@cristianrgreco cristianrgreco added the triage Investigation required label Mar 12, 2023
@cristianrgreco
Copy link
Collaborator

I am integrating podman as a workflow here: #494. I am not sure about officially supporting podman, but already I've found an issue which would prevent tc-node from recognising podman via the DOCKER_HOST, will keep you updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Investigation required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants