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

Use docker or podman compose instead of docker-compose #757

Merged
merged 13 commits into from
Mar 27, 2024
Merged

Conversation

mmatczuk
Copy link
Contributor

podman compose would preconfigure and run docker-compose if it can find it in the path.
It works out of the box and gives us the best of both worlds.

I also tried podman-compose the docker-compose alternative, but the performance is much worse and I failed to fix it in reasonable time.

Enable native Podman builds without fake docker binary.

$ make update-devel-image
STEP 1/8: FROM gcr.io/distroless/static:nonroot
STEP 2/8: COPY LICENSE /licenses/
--> e745ba3153e5
STEP 3/8: COPY LICENSE.3RD_PARTY /licenses/
--> 7b82b58f2e28
STEP 4/8: COPY forwarder /usr/bin
--> 5fe353e9595e
STEP 5/8: ENTRYPOINT ["/usr/bin/forwarder"]
--> 0d7dd897535e
STEP 6/8: CMD ["run"]
--> 9562b65cff9a
STEP 7/8: ENV FORWARDER_API_ADDRESS="localhost:10000"
--> 48741f33c951
STEP 8/8: HEALTHCHECK --interval=1s --timeout=1s --retries=10 CMD ["/usr/bin/forwarder", "ready"]
COMMIT saucelabs/forwarder:devel
--> 88fe020d8de3
Successfully tagged localhost/saucelabs/forwarder:devel
88fe020d8de3955e92caa82ee025a34f087ed799358330d1eedd9b11b23f62b7
Enable container health in status.
@Choraden
Copy link
Contributor

LGTM

time="2024-03-26T14:36:17+01:00" level=warning msg="/Users/michal/Sauce-CONNECT-5/forwarder/e2e/compose.yaml: `version` is obsolete"
Remove --remove-orphans because it's unlikely to have orphans in our usecase.

Add timeout of 1s, the --force-recreate will clean stuff anyway.
@mmatczuk mmatczuk merged commit 1b677b0 into main Mar 27, 2024
4 checks passed
@mmatczuk mmatczuk deleted the mmt/podman2 branch March 27, 2024 09:38
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.

None yet

2 participants