Skip to content

Podman support - #1859

Open
isc-klu wants to merge 34 commits into
intersystems-community:masterfrom
isc-klu:podman-support
Open

Podman support#1859
isc-klu wants to merge 34 commits into
intersystems-community:masterfrom
isc-klu:podman-support

Conversation

@isc-klu

@isc-klu isc-klu commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

I implemented this so I could test docker-compose connection scenarios without installing docker myself. Not sure there's broader interest in podman support — feedback welcome.

Summary

  • composeCommand() falls back to podman-compose when neither docker compose nor docker-compose is present.
  • Added portFromPodmanCompose, used only for podman: its port output is a bare port number (not host:port), and it lacks ps --services --filter status=running, so it needs its own resolution via ps --format json.
  • Docker-compose path is untouched; diff is purely additive aside from one dispatch branch.

Test plan

  • Verified against a real IRIS container under podman-compose: resolves both ports, reports failures correctly.
  • tsc/eslint clean.

isc-klu added 30 commits April 13, 2026 16:31
podman-compose diverges from docker-compose in ways that can't be
papered over by composeCommand alone: its `port` output is a bare
port number instead of a host:port pair, and it doesn't support
`ps --services --filter status=running`. Rather than generalize the
existing docker-compose code to handle both, add a self-contained
portFromPodmanCompose used only when composeCommand resolves to
podman-compose, leaving the original docker-compose logic untouched.
…sitive

The retry-on-parse-error added earlier was speculative and never
confirmed necessary; drop it pending an actual repro.

cmd.includes("podman") could false-positive on a compose file/env
file named with "podman" in it, since cmd also embeds the -f/--env-file
arguments. Check the resolved executable directly instead.
@isc-klu
isc-klu marked this pull request as ready for review September 4, 2026 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant