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

normalize usage of docker-compose #774

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ionous
Copy link
Contributor

@ionous ionous commented Jul 8, 2024

this change makes it so that macos uses the same "dash" compose command as production. ( docker-compose )

this is a "patch" until we can upgrade production. ( re: #772 ) once that's done, then this should be converted to the "space" version ( docker compose )

( even though the dash version is deprecated, the "dash" version works on macos with current versions of docker. the "space" version doesn't yet exist on production )

i've tested this on apple silicon with local Docker version 26.1.1, build 4cf5afa. and i've tested this previously on my older intel mac with ... a relatively recent version of docker.

( eta: production is 24.0.5 - 2023-07-24 )

see also this thread from @carrythebanner

We originally did this docker compose vs docker-compose dance to work around some other failures, but that was in 2021/2022 and Docker v20.10 — see commit notes here:
32d9bd2
7ab6a19

this change makes it so that macos uses the same compose command as production.

this is a "patch" until we can upgrade production. once that's done, then this should be converted to "docker compose" for all platforms. ( the "dash" version is deprecated, but the "space" version doesn't yet exist on production  )

i've tested this on apple silicon with local Docker version 26.1.1, build 4cf5afa. and i've tested this previously on my older intel mac with ... a relatively recent version of docker.
@ionous ionous marked this pull request as draft July 30, 2024 01:57
@ionous
Copy link
Contributor Author

ionous commented Jul 30, 2024

the very latest version of docker removes the docker-compose command.
some clever will be needed until we can upgrade docker on production

@fool
Copy link
Member

fool commented Dec 13, 2024

I'm trying to get some motion on #772 in the coming weeks to unblock this. This feels like a good "off season" upgrade while we can afford to break production :)

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.

2 participants